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

Skip comparison for initial PR, determined by print-config failing #32

Merged
merged 1 commit into from
Jun 19, 2024

Conversation

matejchalk
Copy link
Contributor

@matejchalk matejchalk commented Jun 18, 2024

Fixes #14

Tested initial integration flow on flowup/api-client-generator#141

Copy link

Code PushUp

🤨 Code PushUp report has both improvements and regressions – compared target commit a97d8c8 with source commit 2fb917c.

🏷️ Categories

🏷️ Category ⭐ Current score ⭐ Previous score 🔄 Score change
Code coverage 🟡 78 🟡 77 ↑ +0.1
Bug prevention 🟢 100 🟢 100
Code style 🟢 100 🟢 100
Security 🟢 97 🟢 97
Updates 🟢 99 🟢 99

🗃️ Groups

👍 1 group improved
🔌 Plugin 🗃️ Group ⭐ Current score ⭐ Previous score 🔄 Score change
Code coverage Code coverage metrics 🟡 78 🟡 77 ↑ +0.1

11 other groups are unchanged.

🛡️ Audits

👍 1 audit improved, 👎 2 audits regressed, 1 audit changed without impacting score
🔌 Plugin 🛡️ Audit 📏 Current value 📏 Previous value 🔄 Value change
Code coverage Function coverage 🟨 86 % 🟨 86 %  +0 %
Code coverage Branch coverage 🟨 59 % 🟨 59 %  +0 %
Code coverage Line coverage 🟨 81 % 🟨 81 %  +0 %
JS Packages Outdated NPM dev dependencies. 🟩 8 outdated package versions (1 major, 7 patch) 🟩 1 major outdated package version ↑ +700 %

168 other audits are unchanged.

@matejchalk matejchalk self-assigned this Jun 18, 2024
@matejchalk matejchalk requested a review from vmasek June 18, 2024 15:58
@matejchalk matejchalk merged commit 5990fb9 into main Jun 19, 2024
19 checks passed
@matejchalk matejchalk deleted the fix-initial-flow branch June 19, 2024 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Prevent error on initial integration in PR
2 participants