Skip to content

CI: Don't run cron for forks #53129

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

Merged
merged 1 commit into from
May 8, 2023
Merged

Conversation

hugovk
Copy link
Contributor

@hugovk hugovk commented May 7, 2023

  • [n/a] closes #xxxx (Replace xxxx with the GitHub issue number)
  • [n/a] Tests added and passed if fixing a bug or adding a new feature
  • All code checks passed.
  • [n/a] Added type annotations to new arguments/methods/functions.
  • [n/a?] Added an entry in the latest doc/source/whatsnew/vX.X.X.rst file if fixing a bug or adding a new feature.

The codeql and stale-pr workflows run on a cron:

on:
  schedule:
    # every day at midnight
    - cron: "0 0 * * *"

As does the wheels workflow:

on:
  schedule:
  #        ┌───────────── minute (0 - 59)
  #        │  ┌───────────── hour (0 - 23)
  #        │  │ ┌───────────── day of the month (1 - 31)
  #        │  │ │ ┌───────────── month (1 - 12 or JAN-DEC)
  #        │  │ │ │ ┌───────────── day of the week (0 - 6 or SUN-SAT)
  #        │  │ │ │ │
  - cron: "27 3 */1 * *"

These are useful for the upstream https://github.com/pandas-dev/pandas repo but unnecessary for contributors' forks:

image

They also trigger emails when they fail on the fork:

image

Similar to autoupdate-pre-commit-config, let's only run the schedule for upstream:

if: github.repository_owner == 'pandas-dev'

@mroeschke mroeschke added the CI Continuous Integration label May 8, 2023
@mroeschke mroeschke added this to the 2.1 milestone May 8, 2023
@mroeschke mroeschke merged commit b7e8b47 into pandas-dev:main May 8, 2023
@mroeschke
Copy link
Member

Makes sense. Thanks @hugovk

@hugovk hugovk deleted the no-cron-for-forks branch May 8, 2023 16:38
Rylie-W pushed a commit to Rylie-W/pandas that referenced this pull request May 19, 2023
Daquisu pushed a commit to Daquisu/pandas that referenced this pull request Jul 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants