-
Notifications
You must be signed in to change notification settings - Fork 2
Rename Contile integration tests to contract tests #391
Conversation
8f56d0f
to
6888426
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.
Hi @Trinaa! 👋🏻
Great work! Your changes LGTM. 🚀
Merging is currently blocked because we require all commits on Contile to be signed. There's a GitHub docs page on GPG signing at https://docs.github.com/articles/about-gpg/. |
6888426
to
c812fc4
Compare
@hackebrot I seemed to have fallen into a pitfall, where my signatures end up lost. I'm not sure if forcing signature verification is correct/optimal/allowed |
Hi @Trinaa! 👋🏻 My bad. I wasn't aware of this when I suggested you rebase using the GitHub web UI. I recommend you rebase your local commits to the latest |
c812fc4
to
1679aa5
Compare
Description
This pull request renames the integration-tests directory to contract-tests and updates references in the documentation and CI configuration.
This change clarifies the test suite purpose and distinguishes the suite from the more internal Rust based integration tests.
Issue(s)
CONSVC-1784