Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DO NOT MERGE] Diff md preview #566

Closed
wants to merge 6 commits into from
Closed

[DO NOT MERGE] Diff md preview #566

wants to merge 6 commits into from

Conversation

matejchalk
Copy link
Collaborator

No description provided.

@github-actions github-actions bot added 📖 Project documentation improvements or additions to the project documentation 🔬 testing writing tests 🦾 CI/CD continuous integration and deployment 🧩 utils labels Mar 14, 2024
Copy link

nx-cloud bot commented Mar 14, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 5bce914. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 7 targets

Sent with 💌 from NxCloud.

@matejchalk matejchalk force-pushed the diff-md-preview branch 3 times, most recently from d9e7ac7 to 594ee9c Compare March 14, 2024 11:15
Copy link

Code PushUp

😐 Code PushUp report has both improved and regressed – compared target commit 0123456 with source commit abcdef0.

🏷️ Categories

🏷️ Category ⭐ Current score ⭐ Previous score 🗠 Score change
Performance 🟢 94 🟢 92 +2
Bug prevention 🟡 63 🟡 68 -5
1 other category is unchanged.
  • Code style: 🟡 54

🎗️ Groups

👍 1 group improved
🔌 Plugin 🎗️ Group ⭐ Current score ⭐ Previous score 🗠 Score change
Lighthouse Performance 🟢 94 🟢 92 +2

1 other group is unchanged.

🛡️ Audits

👍 3 audits improved, 👎 1 audit regressed
🔌 Plugin 🛡️ Audit 📏 Current value 📏 Previous value 🗠 Value change
ESLint Disallow unused variables 🟥 1 error 🟩 passed +∞%
Lighthouse First Contentful Paint 🟨 1.1 s 🟨 1.2 s -4%
Lighthouse Largest Contentful Paint 🟨 1.4 s 🟨 1.5 s -8%
Lighthouse Speed Index 🟩 1.1 s 🟩 1.2 s -4%

48 other audits are unchanged.

@matejchalk
Copy link
Collaborator Author

Closing as the feature is now complete and this PR was just for demoing via a simulated PR comment.

@matejchalk matejchalk closed this Mar 20, 2024
@matejchalk matejchalk deleted the diff-md-preview branch March 20, 2024 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🦾 CI/CD continuous integration and deployment 📖 Project documentation improvements or additions to the project documentation 🔬 testing writing tests 🧩 utils
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant