Run the Tests workflow for pull requests into any base branch
Maintainers usually reply within 1 day
Nobody has claimed this yet.
- #1322 by @anth-volk — closed without merging
Assessment
- Difficulty
- 1/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 25/100
Research direction
Read .github/workflows/test.yml and find the test that reads this workflow. Confirm the push trigger remains limited to main, while pull_request has no base branch filter. The issue reports an open linked pull request (#1322), so check its status before starting; done means the workflow and its test cover both trigger settings.
Written by the indexing model from the issue text.
Description
Problem
.github/workflows/test.yml runs for pull_request events only when the pull request's base branch is main:
on:
push:
branches: [main]
pull_request:
branches: [main]
A stacked pull request targets another pull request's branch rather than main. GitHub therefore starts no Tests run for it: lint, engine-free, the country engine jobs, integration-uk and wheels are all skipped, and the pull request shows no checks at all.
We want to stack microcosm#1295, #1297, #1296 and #1298, which build on each other's changes. Without this change, only the bottom pull request of the stack would get CI.
Proposed change
- Remove the
branchesfilter from thepull_requesttrigger, so a pull request into any base branch runs the same jobs. - Leave the
pushtrigger atbranches: [main]. - Add a test that reads the workflow and checks both triggers.
What else in the workflow depends on the base branch
Nothing:
- the concurrency group is keyed on the pull request number;
- country selection (
tools/ci_test_plan.py select-countries) reads the pull request's own changed files from the GitHub API; - the workflow never reads
github.base_ref; - it has no path filters.
Pull requests from branches in this repository get the same GITHUB_TOKEN permissions and the same secrets whatever their base branch. Pull requests from forks get no secrets either way.
- Dominant language
- Python
- Stars
- 0
- Forks
- 6
- Avg merge
- 1d 8h
- Merged PRs (30d)
- 140
Getting set up
- No Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from PolicyEngine/microcosm
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
PolicyEngine/microcosm#1220 ·
Maintainers usually reply within 1 day
-
Update two stale comments about the export-mass reviewed-exclusion registerPossibly taken A pull request linked to this issue is open or already merged. Open
Difficulty 1/5 Under an hour Newbie friendliness 88/100
PolicyEngine/microcosm#1108 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
PolicyEngine/microcosm#475 ·
Maintainers usually reply within 1 day
-
UK: employer NI relief for under-21s is about £0.5-0.8bn below HMRC, beyond missing apprentice flagsOpen
Difficulty 4/5 3-5 days Newbie friendliness 45/100
PolicyEngine/microcosm#1374 ·
Maintainers usually reply within 1 day
-
Impute is_apprentice on the UK spine so the apprentice employer NICs zero rate reaches 21- to 24-year-oldsPossibly taken @vahid-ahmadi claimed this today. Open
Difficulty 4/5 3-5 days Newbie friendliness 42/100
PolicyEngine/microcosm#1372 ·
Maintainers usually reply within 1 day
All issues in PolicyEngine/microcosm
Similar issues
-
[Resource]: LoreOpenresource-submission validation-passed
Difficulty 1/5 Under an hour Newbie friendliness 88/100
hesreallyhim/awesome-claude-code#3234 · 1 comment ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
openminion/openminion#376 ·
Maintainers usually reply within 1 day
-
Type: ☹︎ Bug
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
posit-dev/great-tables#1028 ·
Maintainers usually reply within 1 day
-
integration: apple_tv
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
home-assistant/core#188282 · 1 comment ·
Maintainers usually reply within 1 day
-
A "since" range is worded as a career: "the past two seasons" says "regular season career (2025 on)"OpenP4: low
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
jeffknupp/association#538 ·