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 PAT to clang format #416

Merged
merged 1 commit into from
Jul 12, 2023
Merged

Conversation

k-doering-NOAA
Copy link
Member

What is the feature?

Pass in a Personal access token to clang format.

To use this, we will need an FIMS owner to generate a Personal access token and add it to the FIMS repo as a secret named PAT (if we don't like this name, we can change it!)

How have you implemented the solution?

adding a line to use a PAT in the workflow

Does the PR impact any other area of the project?

This will allow required checks to run on PRS opened from the clang format workflow

Input

n/a

Output

n/a

How to test this change

After putting the PAT in secrets and merging this to main, the workflow should run and generate a PR. This PR should run additional workflows, in particular r cmd check.

Developer pre-PR checklist

  • I relied on GitHub actions to 🧪 things for me while I sat on the 🛋️.

@ChristineStawitz-NOAA ChristineStawitz-NOAA merged commit 0c34af0 into main Jul 12, 2023
9 checks passed
@ChristineStawitz-NOAA ChristineStawitz-NOAA deleted the add-pat-to-clang-format branch July 12, 2023 13:43
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