Add minimal CI workflow placeholder on master - #4
Merged
Merged
Conversation
Wires up GitHub Actions to fire on push and pull_request against master. The job is intentionally a no-op for now — feature-branch PRs (e.g. the modernization branch) ship their own ci.yml that overrides this with the real build/test/lint pipeline. Without this placeholder on master, PRs opened against master never trigger any CI run. https://claude.ai/code/session_01Gqf38F1k7dPDMQCHJzNzHU
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
.github/workflows/ci.ymlso GitHub Actions fires on push and pull_request againstmasterci.ymlthat overrides this with the real build/test/lint pipelineWhy
Without a workflow file on
master, the default branch is "CI-unaware." The modernization PR (#3) targetsmaster, but its workflow'spull_requesttrigger originally listed[main], so it silently never fired. Landing this minimal placeholder onmasterensures future PRs always trigger CI even before they bring their own workflow changes.Test plan
masterafter merge[master]trigger fix is in placehttps://claude.ai/code/session_01Gqf38F1k7dPDMQCHJzNzHU
Generated by Claude Code