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

Create a workflow for maintaining a list of labeled tests in-repo #288

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

foolip
Copy link
Member

@foolip foolip commented Feb 16, 2023

No description provided.

@foolip foolip marked this pull request as ready for review February 16, 2023 10:11
@foolip foolip requested a review from jgraham February 23, 2023 14:03
@gsnedders gsnedders self-requested a review March 15, 2023 19:24
@foolip foolip mentioned this pull request Mar 16, 2023
on:
# Trigger every Monday morning, or manually.
schedule:
- cron: 55 5 * * 1
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Scheduled workflows are disabled by Github after 60 days without activity on the repository. will this repo have sufficient activity so that this does not happen or should the scheduled workflow be in the wpt repo, using a workflow dispatch to trigger the workflow on this repo ?
See DefinitelyTyped/DefinitelyTyped#64327 and DefinitelyTyped/dt-mergebot#457

Note that instead of a personal access token with a repo scope (which grants full access to the repo) stored as a secret, it might be possible to define a GitHub App installed in this interop repo and having the actions:write permission. The wpt repo would then have a secret containing the id and secret key of that app to be able to get a temporary token to authenticate as that app using https://github.com/marketplace/actions/create-github-app-token (listing the interop repo as part of the repositories accessible by that token).

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.

2 participants