-
Notifications
You must be signed in to change notification settings - Fork 24
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
Add C++20 three-way comparison operator #49
Add C++20 three-way comparison operator #49
Conversation
Here is what the Test workflow looks like : It initiates the work depicted in #44 |
077f6f2
to
6382590
Compare
6382590
to
c8e7b64
Compare
The new operator, also known as `spaceship operator` was introduced in the C++20 standard. The NSIQ parser now handles it properly.
The commit 430a4e7 changed the rule processing to include comment rules. The RULE_4_1_A_A expected to NOT run on comments. The rule was updated to exclude comment lines.
The github action will run both test-suites: - rules - nsiqunittest
@kunaltyagi do you plan on merging this at some point by any chance? |
@Farigh I'm a contributor, couldn't find your email, drop me a line at [email protected] |
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.
Thanks for the neat update to tests @Farigh
The new operator, also known as the spaceship operator, was introduced in the C++20 standard.
The NSIQ parser now handles it properly.
I fixed some tests as well in this PR and added a GitHub Action to ensure this is performed for each PR from now on.
I greatly encourage you to enforce Checks on PR merge, this can be found in the branch protection settings :