Skip to content
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

Add a test to ensure that #6288 is reverted in v1.22 #6430

Closed
wants to merge 6 commits into from

Conversation

jgsogo
Copy link
Contributor

@jgsogo jgsogo commented Jan 28, 2020

Replace #6292

Changelog: omit
Docs: omit

This test ensures that changes introduced in #6288 won't be deployed in Conan v1.22:

  • The test fails with changes introduced in that PR
  • The test fails if the version is >=1.22.0

Rationale: #6269 (comment)

@jgsogo
Copy link
Contributor Author

jgsogo commented Jan 28, 2020

After merging #6429, this PR should pass (we can then safely close it)

@jgsogo jgsogo added this to the 1.22 milestone Jan 28, 2020
@jgsogo jgsogo self-assigned this Jan 28, 2020
@memsharded
Copy link
Member

I think this can be closed now that #6429 has been merged

@memsharded memsharded closed this Jan 28, 2020
@jgsogo
Copy link
Contributor Author

jgsogo commented Jan 28, 2020

Let's do it the right way, let's check that after merging develop, the tests pass

@jgsogo jgsogo reopened this Jan 28, 2020
@memsharded
Copy link
Member

Let's do it the right way, let's check that after merging develop, the tests pass

Oh, I already did that. I merged develop and ran the tests locally, it was all good. I just didn't want to burn more CI cycles for such a quick test.

Good, it seems CI says it is green, we can close it now.

@memsharded memsharded closed this Jan 28, 2020
@jgsogo jgsogo deleted the sentinel/cmake-generator-1.22 branch January 29, 2020 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants