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

Disable GHA workflows on non-SciTools branches #4444

Merged
merged 5 commits into from
Dec 1, 2021

Conversation

wjbenfold
Copy link
Contributor

@wjbenfold wjbenfold commented Dec 1, 2021

🚀 Pull Request

Description

Prevents GH actions from running automatically in forks. Fixes #4441

Reviewer considerations:

  • Will this have adverse effects on any GH workflows (e.g. making releases) or does this all happen in the main repo?

Consult Iris pull request check list

@wjbenfold wjbenfold changed the title WIP Prevent automatic refresh lockfiles Prevent automatic refresh lockfiles Dec 1, 2021
@wjbenfold
Copy link
Contributor Author

Once I'd ironed some bugs out, the following links now show CI running on Scitools/iris when I make commits but not on wjbenfold/iris (also no syntax errors reported by GitHub). It's a bit of a leap of faith that the cron-like ones will run on Scitools/iris, but we may only be able to test that in live?

https://github.com/SciTools/iris/actions
https://github.com/wjbenfold/iris/actions

@wjbenfold wjbenfold requested a review from jamesp December 1, 2021 12:04
@jamesp jamesp changed the title Prevent automatic refresh lockfiles Disable GHA workflows on non-SciTools branches Dec 1, 2021
@jamesp jamesp merged commit 63ec393 into SciTools:main Dec 1, 2021
tkknight added a commit to tkknight/iris that referenced this pull request Dec 7, 2021
* main:
  [pre-commit.ci] pre-commit autoupdate (SciTools#4452)
  Whatsnew (SciTools#4451)
  Explicitly define tests so nose can find them (SciTools#4450)
  Updated environment lockfiles (SciTools#4449)
  Update CI environment lockfiles (SciTools#4424)
  Disable GHA workflows on non-SciTools branches (SciTools#4444)
  Add new contributor to whatsnew (SciTools#4443)
  Use dask-core instead of dask in ci (SciTools#4434)
  Mesh recombine (SciTools#4437)
  Mesh full comparison (SciTools#4439)
  Only try to work out the differences between points for multiple (SciTools#4367)
  [pre-commit.ci] pre-commit autoupdate (SciTools#4430)
  Fix license PyPI classifier (SciTools#4435)
  Whatsnew for PR SciTools#4367 (SciTools#4440)
@wjbenfold wjbenfold deleted the wjbenfold-stop-default-gha branch December 14, 2021 16:09
tkknight added a commit to tkknight/iris that referenced this pull request Dec 14, 2021
* upstream/main: (78 commits)
  Updated environment lockfiles (SciTools#4458)
  remove asv package dependency (SciTools#4456)
  cube.aggregated_by output bounds (SciTools#4315)
  [pre-commit.ci] pre-commit autoupdate (SciTools#4452)
  Whatsnew (SciTools#4451)
  Explicitly define tests so nose can find them (SciTools#4450)
  Updated environment lockfiles (SciTools#4449)
  Update CI environment lockfiles (SciTools#4424)
  Disable GHA workflows on non-SciTools branches (SciTools#4444)
  Add new contributor to whatsnew (SciTools#4443)
  Use dask-core instead of dask in ci (SciTools#4434)
  Mesh recombine (SciTools#4437)
  Mesh full comparison (SciTools#4439)
  Only try to work out the differences between points for multiple (SciTools#4367)
  [pre-commit.ci] pre-commit autoupdate (SciTools#4430)
  Fix license PyPI classifier (SciTools#4435)
  Whatsnew for PR SciTools#4367 (SciTools#4440)
  Suggest type hinting (SciTools#4390)
  area weight regrid test fixes (SciTools#4432)
  Update latest.rst (SciTools#4425)
  ...
@trexfeathers trexfeathers mentioned this pull request Jun 16, 2022
6 tasks
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.

Stop GHA running in forks?
2 participants