New tidy checks #857
Merged
New tidy checks #857
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This commit adds two new checks for port connections: NoDotStarInPortConnection: Checks if the syntax .* has been used to connect ports NoImplicitPortNameInPortConnection: Checks if the syntax .port_name has been used to connect ports
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #857 +/- ##
========================================
Coverage 93.71% 93.72%
========================================
Files 191 191
Lines 46552 46673 +121
========================================
+ Hits 43626 43743 +117
- Misses 2926 2930 +4 see 15 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Two new checks for slang-tidy:
NoDotStarInPortConnection: Checks if the syntax .* has been used to
connect ports
NoImplicitPortNameInPortConnection: Checks if the syntax .port_name has
been used to connect ports