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
core: Extract AttrParser out of Parser #1211
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #1211 +/- ##
==========================================
- Coverage 89.01% 88.93% -0.08%
==========================================
Files 174 175 +1
Lines 23545 23585 +40
Branches 3560 3567 +7
==========================================
+ Hits 20958 20975 +17
- Misses 2020 2043 +23
Partials 567 567
☔ View full report in Codecov by Sentry. |
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.
I'm surprised. Don't we have parser specific tests? Shouldn't those be split as well? o/w LGTM.
Right now, |
(I'm waiting for #1210 to be merged to merge this, as these are conflicting) |
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.
👍
fe76405
to
6bbde3a
Compare
824f2c1
to
a6909a2
Compare
This PR moves the attribute parsing functions out of Parser, and moves them to a new
AttrParser
.AttrParser
is now given to attributes and types, so all the unrelated methods are not shown anymore in the IDE.Also, this cleans up a bit the file as we now have two smaller files instead of a huge one.
There are no changes in the methods moved to
AttrParser
, so no need to review the content of the methods moved.