Skip to content

build(#131): use app token to authenticate workflow - #132

Merged
BBeltz1 merged 3 commits into
devfrom
build/i131-github-app
May 27, 2026
Merged

build(#131): use app token to authenticate workflow#132
BBeltz1 merged 3 commits into
devfrom
build/i131-github-app

Conversation

@andybeet

@andybeet andybeet commented May 26, 2026

Copy link
Copy Markdown
Member

Your commits explain the who, what, where and when of these changes. Your code shows the how. You do not need to reiterate this. This PR should complete the picture by explaining why these changes are necessary.
Please complete the fields below, replacing or removing placeholder text where necessary:

Justification

With the creation of repository rulesets protecting from directly committing to main or dev. The getFred workflow can not pull and commit to main on its regular cycle. A GitHub App called auto-commit-to-main has been created in the NOAA-EDAB org. And secrets added to this repo to be used in the yml.

Fixes #131

Types of changes

What types of changes does this pull request introduce? Put an x in the boxes that apply.
This will inform the new release number.

  • Fix (non-breaking change which fixes a bug)
  • Feature (non-breaking change which adds or changes functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Other change (if none of the other choices apply)

Reviewer instructions

Not sure how this can be tested prior to merging into main.
It works on a forked version of this repo.

Formatting

This repo contains an air.toml file that automatically formats code to a set of standards.
It is preferred that contributors and reviewers install the Air formatting tool.
Code submitted in this pull request will be automatically checked for correct formatting.

@andybeet
andybeet marked this pull request as draft May 26, 2026 20:16
@andybeet
andybeet marked this pull request as ready for review May 27, 2026 14:13
@andybeet
andybeet requested a review from BBeltz1 May 27, 2026 18:29

@BBeltz1 BBeltz1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

the code looks good to me. if this worked on the fork, i think its safe to approve. we can tweak after deployment if needed

@BBeltz1
BBeltz1 merged commit 2f85ac1 into dev May 27, 2026
10 checks passed
@BBeltz1
BBeltz1 deleted the build/i131-github-app branch May 27, 2026 18:33
@BBeltz1 BBeltz1 linked an issue May 27, 2026 that may be closed by this pull request
@andybeet andybeet changed the title build(github-app): use app token to authenticate build(#131): use app token to authenticate workflow May 27, 2026
@andybeet andybeet mentioned this pull request May 27, 2026
6 tasks
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.

update workflow to use github app

2 participants