Skip to content

Update GitHub Action - #699

Merged
jhaaaa merged 2 commits into
mainfrom
nick-updt-gh-action
Aug 5, 2025
Merged

jhaaaa merged 2 commits into
mainfrom
nick-updt-gh-action

Conversation

@jhaaaa

@jhaaaa jhaaaa commented Aug 5, 2025

Copy link
Copy Markdown
Contributor

Recreating @neekolas's PR with his original commit only: 5f2309c

Closing his original PR, which @jhaaaa corrupted: #680

@jhaaaa
jhaaaa requested a review from a team as a code owner August 5, 2025 19:54
@changeset-bot

changeset-bot Bot commented Aug 5, 2025

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: f693d9c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@macroscopeapp

macroscopeapp Bot commented Aug 5, 2025

Copy link
Copy Markdown

Update GitHub Action workflows to use custom release token and add customizable PR creation in documentation workflow

  • Changes the GitHub token in release.yml from secrets.GITHUB_TOKEN to secrets.RELEASE_GH_TOKEN
  • Modifies update-docs.yml to support custom PR titles and descriptions through claude_pr_title.txt and claude_pr_body.txt files, adds branch cleanup logic to prevent conflicts, and removes automatic labeling from created PRs

📍Where to Start

Start with the token change in release.yml and then review the PR creation logic modifications in update-docs.yml.

Changes since #699 opened

  • Added whitespace formatting between conditional blocks in GitHub workflow [f693d9c]

Macroscope summarized f693d9c.

Comment thread .github/workflows/update-docs.yml
@jhaaaa jhaaaa mentioned this pull request Aug 5, 2025
@jhaaaa
jhaaaa merged commit a5a3109 into main Aug 5, 2025
6 of 7 checks passed
@jhaaaa
jhaaaa deleted the nick-updt-gh-action branch August 5, 2025 19:59

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Issue on line in .github/workflows/update-docs.yml:221:

The default pull request body is redefined immediately after its initial assignment, causing the first set of content to be discarded and leading to the loss of custom default text.
This happens because there are two consecutive PR_BODY= assignments in the else block, so the second one overwrites the first.
Consider removing the duplicate PR_BODY= assignment so that the PR body includes all intended content without being overwritten.

-           PR_BODY="## 🤖 Automated React Native SDK Documentation Update

🚀 Reply to ask Macroscope to explain or update this suggestion.

👍 Helpful? React to give us feedback.

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