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

Add shared configs #1

Merged
merged 2 commits into from
May 7, 2021
Merged

Add shared configs #1

merged 2 commits into from
May 7, 2021

Conversation

narwold
Copy link
Contributor

@narwold narwold commented May 4, 2021

PRs

Add shared configs option to labeler action (this PR)
Add shared labels to .github
Add labels to authentication-server
Add labels to js-frontend

Context

This PR adds the ability for the labeler action to accept remote shared configs so that org-wide label patterns can be applied to any given repo.

Validation

Validate with https://github.com/beamtech/authentication-server/pull/465

How to QA

dist/index.js Outdated Show resolved Hide resolved
tsconfig.json Outdated Show resolved Hide resolved
src/main.ts Show resolved Hide resolved
Copy link
Member

@silasb silasb left a comment

Choose a reason for hiding this comment

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

Looks good. Going to issue a PR against the upstream repo to see if we can avoid a forked version.

@narwold narwold force-pushed the add-shared-configs branch from 63cdcac to cab8ffd Compare May 7, 2021 19:51
@narwold
Copy link
Contributor Author

narwold commented May 7, 2021

Looks good. Going to issue a PR against the upstream repo to see if we can avoid a forked version.

actions/labeler#141

@narwold narwold force-pushed the add-shared-configs branch from 8654499 to 0077fb0 Compare May 7, 2021 21:13
@narwold narwold force-pushed the add-shared-configs branch from 14fb891 to 968a249 Compare May 7, 2021 21:22
@narwold narwold merged commit 559d6d8 into main May 7, 2021
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.

3 participants