-
Notifications
You must be signed in to change notification settings - Fork 2.3k
tests(tools): add unit tests for astfmtgen tool #19073
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
base: main
Are you sure you want to change the base?
Conversation
Review ChecklistHello reviewers! 👋 Please follow this checklist when reviewing this Pull Request. General
Tests
Documentation
New flags
If a workflow is added or modified:
Backward compatibility
|
Signed-off-by: Anuj Agrawal <[email protected]>
e5de7a5 to
34fef00
Compare
|
@timvaillancourt PTAL |
|
@anujagrawal380 you have failing tests. You should make sure that all tests are passing before asking for reviews. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #19073 +/- ##
==========================================
+ Coverage 69.90% 69.95% +0.04%
==========================================
Files 1612 1612
Lines 215796 215817 +21
==========================================
+ Hits 150858 150968 +110
+ Misses 64938 64849 -89 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Description
This PR adds comprehensive test coverage for the
astfmtgentool, which is responsible for generating optimized AST formatting code.Issue