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

updating segalign #725

Merged
merged 2 commits into from
Apr 19, 2024
Merged

updating segalign #725

merged 2 commits into from
Apr 19, 2024

Conversation

richard-burhans
Copy link
Contributor

TODO: Please replace this header with a description of your pull request.

Installation sequence for tool-installers

  • Test using @galaxybot test this
  • Inspect CI output for expected changes
  • Deploy using @galaxybot deploy this if test install was successful
  • Merge this PR

@richard-burhans richard-burhans requested a review from a team as a code owner April 18, 2024 17:00
@martenson
Copy link
Member

@galaxybot test this

@martenson
Copy link
Member

@galaxybot test this

1 similar comment
@natefoo
Copy link
Member

natefoo commented Apr 18, 2024

@galaxybot test this

@martenson
Copy link
Member

did we uninstall the previous version or why are we replacing the changeset here instead of adding?

@natefoo
Copy link
Member

natefoo commented Apr 18, 2024

I'll uninstall the previous version (manually) and removing it here prevents it from being reinstalled later. That one is running in to this bug and so will never run properly on Main unless we fix Pulsar.

@natefoo
Copy link
Member

natefoo commented Apr 18, 2024

@galaxybot deploy this

@martenson martenson merged commit 7a2fecc into galaxyproject:master Apr 19, 2024
1 check passed
@martenson
Copy link
Member

new version is up, (the old one still there too)

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.

3 participants