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

Consider not pinging issues on the weekend #201

Closed
thatch opened this issue Jun 6, 2022 · 2 comments · Fixed by #230
Closed

Consider not pinging issues on the weekend #201

thatch opened this issue Jun 6, 2022 · 2 comments · Fixed by #230

Comments

@thatch
Copy link

thatch commented Jun 6, 2022

For example google/makerspace-auth#47 -- it's existed in this state for years, and I get github notifications on my personal devices. This is not kind to work-life balance, if this is intended for work projects...

@ethan7g
Copy link
Contributor

ethan7g commented Jun 29, 2022

This would be a nice org + repo config option.
Weekends could be configured sort of like this:

optOutDays:
- issue: false
  pr: false
  days:
    - 0
    - 6
  timezone: pst

See Weekday.
Limit the config to opting out three days per week or so.

@ethan7g
Copy link
Contributor

ethan7g commented Jun 30, 2022

As discussed in today's bi-weekly meeting, it might be nice to take some inspiration from the way Dependabot's config handles skipping days (see relevant Dependabot docs here).

Here is what that might look like in an organization's allstar.yaml config:

optConfig:
  optOutStrategy: true
  optSchedule:
    timezone: America/Los_Angeles
    actions:
      issue: false
      fix: true
    days:
      - saturday
      - sunday

Fix will default to true.

It isn't exactly the same, but it is familiar. We could also automatically support the Dependabot configuration by looking at each repo's .github/dependabot.yml to find the schedule information there, inverting the days listed to get skipDays.

Be sure to limit skipDays to 3 days or so so that it isn't used as a tricky opt-out strategy.

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 a pull request may close this issue.

2 participants