-
Notifications
You must be signed in to change notification settings - Fork 23
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
Fix assign stmt #2143
Fix assign stmt #2143
Conversation
); | ||
|
||
always_comb begin | ||
logic temp = 0; |
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.
Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
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 believe this hasn't been addressed
logic temp = 0; | |
logic temp = 0; |
temp = b; | ||
end | ||
|
||
endmodule |
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.
Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
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.
similar here
endmodule | |
endmodule |
LGTM, @alaindargelas can you address linter comments, so we can merge this? |
@alaindargelas you've resolved the linter bot without actually addressing the issues? |
@alaindargelas please fix the trailing spaces issues pointed out by the linter so we can merge this. Also, this PR needs rebasing as Surelog submodule has been bumped by dependabot in the meantime |
No description provided.