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 note for organisation usage #538

Merged
merged 1 commit into from
Nov 12, 2022
Merged

Add note for organisation usage #538

merged 1 commit into from
Nov 12, 2022

Conversation

0xTim
Copy link
Contributor

@0xTim 0xTim commented Nov 9, 2022

Hi,

Thanks for this action! I've added a note to the README to alert users than anyone using it for an org must provide a PAT from a user that is an org owner otherwise it will fail. This tripped us up for a bit in Vapor so might be useful to note!

@0xTim 0xTim requested a review from JamesIves as a code owner November 9, 2022 23:40
@JamesIves
Copy link
Owner

Thanks! Glad you got it working too!

@JamesIves JamesIves merged commit 7e9f264 into JamesIves:dev Nov 12, 2022
@0xTim 0xTim deleted the patch-1 branch January 18, 2023 09:57
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.

None yet

2 participants