Skip to content

Conversation

ashutosh-ukey
Copy link
Contributor

@ashutosh-ukey ashutosh-ukey commented Dec 24, 2024

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

  • Did you add new tests and confirm all tests pass? (yarn test)
  • [] Did you update relevant docs? (docs are found in the docs folder)
  • Do your commits follow the Conventional Commits standard?
  • Does your PR title also follow the Conventional Commits standard?
  • If you have a breaking change, is it correctly reflected in your commit message? (e.g. feat!: breaking change)
  • Did you run lint (yarn lint) and fix any issues?
  • [] Did you run formatter (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.

// TODO: uncomment line once source code coverages have been bumped up
// process.exit(2);
}
})(); No newline at end of file
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missing new line

Copy link
Contributor

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?

Copy link
Contributor

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

Copy link
Contributor

@huaweigu huaweigu left a 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?

@ashutosh-ukey
Copy link
Contributor Author

@huaweigu yes, should be good to merge 👍

@huaweigu huaweigu merged commit 517b89a into circlefin:master Feb 3, 2025
6 checks passed
huaweigu pushed a commit that referenced this pull request Feb 21, 2025
🤖 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).
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.

2 participants