Skip to content

Limit PR checks to building the rock#58

Merged
dwilding merged 3 commits intocanonical:mainfrom
dwilding:rock-only-on-pr
Sep 12, 2025
Merged

Limit PR checks to building the rock#58
dwilding merged 3 commits intocanonical:mainfrom
dwilding:rock-only-on-pr

Conversation

@dwilding
Copy link
Collaborator

@dwilding dwilding commented Sep 12, 2025

Previously, the publish.yaml workflow was set to run on every PR, which caused a lot of unnecessary building and publishing. For PRs, I think we only need to build the rock, as PRs almost always touch the Django application only. We can still run the publish workflow manually for a PR if we want.

This PR:

  • Changes the publish.yaml workflow so that it only runs on push to main
  • Adds a rock.yaml workflow that builds the rock, only running on PRs

@dwilding dwilding marked this pull request as ready for review September 12, 2025 02:49
@dwilding dwilding merged commit 0006e8f into canonical:main Sep 12, 2025
2 checks passed
@dwilding dwilding deleted the rock-only-on-pr branch September 12, 2025 04:31
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