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

DM-48404 : Refactor Github Actions #156

Merged
merged 1 commit into from
Jan 15, 2025
Merged

Conversation

tcjennings
Copy link
Contributor

Splits GitHub actions into separate workflows:

CI - runs lint, typing, and tests for any push to an active branch.

Release - runs python-semantic-release when a PR is merged to main. This includes bumping the project version and writing a tag.

Build-Push - performs docker container build operations when a tagged release is made or when an active ticket branch is part of a PR.

@tcjennings tcjennings force-pushed the tickets/DM-48099/v0.2.0 branch from e6bcb19 to 1307807 Compare January 10, 2025 22:50
@tcjennings tcjennings force-pushed the tickets/DM-48099/gh_actions branch 3 times, most recently from bf693d4 to 39c9d95 Compare January 13, 2025 20:39
@tcjennings tcjennings changed the base branch from tickets/DM-48099/v0.2.0 to main January 13, 2025 20:40
@tcjennings tcjennings changed the base branch from main to tickets/DM-48099/v0.2.0 January 13, 2025 20:41
@tcjennings tcjennings changed the title DM-48099 : Refactor Github Actions DM-48404 : Refactor Github Actions Jan 13, 2025
@tcjennings tcjennings force-pushed the tickets/DM-48099/v0.2.0 branch from cc3d442 to 016065a Compare January 14, 2025 14:46
@tcjennings tcjennings force-pushed the tickets/DM-48099/gh_actions branch from 39c9d95 to 0777889 Compare January 14, 2025 14:46
@tcjennings tcjennings changed the base branch from tickets/DM-48099/v0.2.0 to main January 14, 2025 14:49
@tcjennings tcjennings force-pushed the tickets/DM-48099/gh_actions branch from 0777889 to a09f00e Compare January 14, 2025 15:13
needs:
- ci
runs-on: ubuntu-latest
timeout-minutes: 20
Copy link

Choose a reason for hiding this comment

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

Is there any reasonable situation in which 20 minutes wouldn't be enough? Probably not but worth thinking about for a minute if you didn't already.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think this is an inherited value from ultimately a square template, but it is a reasonable value if a bit long. 20 minutes would be an eternity for a build.

@tcjennings tcjennings force-pushed the tickets/DM-48099/gh_actions branch from a09f00e to 90161ac Compare January 15, 2025 20:49
@tcjennings tcjennings merged commit a24381e into main Jan 15, 2025
4 checks passed
@tcjennings tcjennings deleted the tickets/DM-48099/gh_actions branch January 15, 2025 20:53
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