-
Notifications
You must be signed in to change notification settings - Fork 28
ci: add code coverage checking script #58
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
ci: add code coverage checking script #58
Conversation
// TODO: uncomment line once source code coverages have been bumped up | ||
// process.exit(2); | ||
} | ||
})(); No newline at end of file |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing new line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we please fix this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually you can fix this when the threshold is enforced
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ashutosh-ukey Is this PR good to go?
@huaweigu yes, should be good to merge 👍 |
🤖 I have created a release *beep* *boop* --- ## [2.2.0](v2.1.2...v2.2.0) (2025-02-20) ### Features * add bytecode deployment scripts ([#68](#68)) ([a0fdf60](a0fdf60)) * sync the latest development ([#72](#72)) ([4e6281f](4e6281f)) ### Miscellaneous Chores * add missing copyright header ([#70](#70)) ([431adb9](431adb9)) ### Continuous Integration * add code coverage checking script ([#58](#58)) ([517b89a](517b89a)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Summary
Add code coverage checking script.
Detail
Add a script that checks that the coverage of each files matches our minimum defined threshold. Currently, the script doesn't fail. Once you files meet our defined threshold, we can uncomment the line tht will make the script start failing.
Checklist
yarn test
)docs
folder)feat!: breaking change
)yarn lint
) and fix any issues?yarn format:check
) and fix any issues (yarn format:write
)?Testing
See
Coverage -> run_coverage (pull_request) -> Run Test Coverage
job logs in latest PR build.