Skip to content

brighterscript-formatter 1.8.0#8693

Closed
chenrui333 wants to merge 1 commit into
mainfrom
bump-brighterscript-formatter-1.8.0
Closed

brighterscript-formatter 1.8.0#8693
chenrui333 wants to merge 1 commit into
mainfrom
bump-brighterscript-formatter-1.8.0

Conversation

@chenrui333

Copy link
Copy Markdown
Owner

Created by brew bump


Created with brew bump-formula-pr.

Copilot AI review requested due to automatic review settings June 30, 2026 18:56
@github-actions github-actions Bot added the nodejs Node or npm use is a significant feature of the PR or issue label Jun 30, 2026

Copilot AI 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.

Pull request overview

Updates the brighterscript-formatter Homebrew formula to the latest npm tarball release, consistent with a brew bump-formula-pr version bump workflow.

Changes:

  • Bumped brighterscript-formatter from 1.7.28 to 1.8.0.
  • Updated the source tarball url and corresponding sha256 checksum.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 31514ab251

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +4 to +5
url "https://registry.npmjs.org/brighterscript-formatter/-/brighterscript-formatter-1.8.0.tgz"
sha256 "7077c67b84044a4c8978d645c868495ef567da857f07f763c8337c896abe0c07"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Bump to the regression-fix release

This points Homebrew users at 1.8.0, but upstream has already released 1.8.1 specifically to fix a 1.8.0 formatter regression where multi-line calls whose continuation line ends with ) are de-indented back to the call level (rokucommunity/brighterscript-formatter#147). In that scenario this formula would ship incorrect formatting behavior; please update the URL and checksum to 1.8.1 instead.

Useful? React with 👍 / 👎.

@github-actions github-actions Bot closed this in c3d653b Jun 30, 2026
@github-actions github-actions Bot deleted the bump-brighterscript-formatter-1.8.0 branch June 30, 2026 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

nodejs Node or npm use is a significant feature of the PR or issue pr-pull

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants