-
Notifications
You must be signed in to change notification settings - Fork 28
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
Adding GitHub Actions for DRC testing #11
Conversation
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.
Same question about sram in sc_testcases
directory?
@umarcor - I would appreciate your thoughts on this pull request. |
@mohanad0mohamed Could you please change the target name to match what @mithro has requested? I agree that it would make the target more readable and understandable. |
@mithro This is not a complete DRC regression. But merging this version is very important. There are 2 updates that will be done that will make the regression more robust and able to detect changes that might affect the DRC quality. |
c33cc53
to
0fbc1c3
Compare
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 approve the changes made in this PR.
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.
Updating primitves names in lvs rule deck and its README file
Fixes #3
All actions required for DRC are now running, but needs this PR #10 to be merged first.