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

Remove showmigrations from CI / Release pipelines #224

Merged
merged 2 commits into from
Dec 1, 2022

Conversation

rajadain
Copy link
Contributor

@rajadain rajadain commented Dec 1, 2022

Overview

This step was not used and caused significant delays to CI jobs. For example, see here:

https://github.com/azavea/iow-boundary-tool/actions/runs/3566198623/jobs/5992333569

The deployment takes ~28s, and showmigrations took 1m41s.

Checklist

  • fixup! commits have been squashed
  • CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines
  • README.md updated if necessary to reflect the changes
  • CI passes after rebase

This step was not used and caused significant delays to CI jobs.
For example, see here:

https://github.com/azavea/iow-boundary-tool/actions/runs/3566198623/jobs/5992333569

The deployment takes ~28s, and showmigrations took 1m41s.
Copy link
Contributor

@JN-Hernandez JN-Hernandez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks for tackling this so quickly! +1

@rajadain rajadain merged commit 2efffb6 into develop Dec 1, 2022
@rajadain rajadain deleted the tt/remove-showmigrations branch December 1, 2022 21:09
@rajadain
Copy link
Contributor Author

rajadain commented Dec 2, 2022

Couldn't wait to get rid of it

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