-
Notifications
You must be signed in to change notification settings - Fork 108
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
xc7: tests: reformat tests to use common pcf files #1234
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.
I like it how all the tests are now consistent. Let's merge provided you didn't make any typos and CI is green.
f581775
to
235522d
Compare
Signed-off-by: Alessandro Comodi <acomodi@antmicro.com>
Signed-off-by: Alessandro Comodi <acomodi@antmicro.com>
Signed-off-by: Alessandro Comodi <acomodi@antmicro.com>
Signed-off-by: Alessandro Comodi <acomodi@antmicro.com>
Signed-off-by: Alessandro Comodi <acomodi@antmicro.com>
Some of the tests require specialized pcf files as they use uncommon IOs Signed-off-by: Alessandro Comodi <acomodi@antmicro.com>
235522d
to
a307a4b
Compare
CI went red only on vendor tool tests, with the same failure experienced in other PRs and unrelated to this one. I believe this can be merged |
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.
LGTM
Signed-off-by: Alessandro Comodi acomodi@antmicro.com
This PR is to clean all the xc7 tests, by using the common pcf files.
Some of the tests need a custom
.pcf
file, but the majority of them can adopt the one found inxc7/tests/common