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

Use pre-releases for Fortress #35

Merged
merged 1 commit into from
Sep 21, 2021
Merged

Conversation

chapulina
Copy link
Contributor

@chapulina chapulina commented Sep 21, 2021

Removed nightlies from all libs above ign-gazebo, and appended it for the ones below

gazebo-tooling/release-tools#459

Signed-off-by: Louise Poubel <[email protected]>
@azeey
Copy link
Contributor

azeey commented Sep 21, 2021

Removed nightlies from all libs above ign-gazebo, and appended it for the ones below

Not sure what above and below mean in this context, would it not work if all of them had all three repos?

@chapulina
Copy link
Contributor Author

Not sure what above and below mean in this context,

Sorry, by above I meant upstream. We already have pre-releases up to ign-gui and ign-physics, so the only libs that still need nightlies are ign-gazebo, ign-launch and ignition-fortress.

would it not work if all of them had all three repos?

It may, but mixing up pre-releases and nightlies may have unintended consequences that I'd like to avoid as much as possible

@azeey
Copy link
Contributor

azeey commented Sep 21, 2021

I see. Thanks for the explanation.

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