Skip to content
This repository has been archived by the owner on Jul 26, 2024. It is now read-only.

Rename Contile integration tests to contract tests #391

Merged
merged 3 commits into from
Jun 3, 2022

Conversation

Trinaa
Copy link
Collaborator

@Trinaa Trinaa commented May 26, 2022

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

@Trinaa Trinaa requested a review from hackebrot May 26, 2022 18:21
@ncloudioj
Copy link
Collaborator

Hey @Trinaa, so #394 is now closed. A rebase should green your PR.

@Trinaa Trinaa force-pushed the rename-integration-to-contract-tests branch from 8f56d0f to 6888426 Compare June 2, 2022 13:50
@Trinaa Trinaa marked this pull request as ready for review June 2, 2022 13:50
pjenvey
pjenvey previously approved these changes Jun 2, 2022
hackebrot
hackebrot previously approved these changes Jun 2, 2022
Copy link
Member

@hackebrot hackebrot left a 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. 🚀

@hackebrot
Copy link
Member

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/.

@Trinaa Trinaa dismissed stale reviews from hackebrot and pjenvey via c812fc4 June 2, 2022 19:29
@Trinaa Trinaa force-pushed the rename-integration-to-contract-tests branch from 6888426 to c812fc4 Compare June 2, 2022 19:29
@Trinaa
Copy link
Collaborator Author

Trinaa commented Jun 2, 2022

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/.

@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

@hackebrot
Copy link
Member

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 origin/main and force-push your branch.

@Trinaa Trinaa force-pushed the rename-integration-to-contract-tests branch from c812fc4 to 1679aa5 Compare June 3, 2022 12:52
@Trinaa Trinaa requested a review from hackebrot June 3, 2022 12:53
@Trinaa Trinaa merged commit e4d9331 into main Jun 3, 2022
@Trinaa Trinaa deleted the rename-integration-to-contract-tests branch June 3, 2022 14:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants