Skip to content

Conversation

@ljunkie
Copy link
Member

@ljunkie ljunkie commented Nov 29, 2025

No description provided.

@ljunkie ljunkie self-assigned this Nov 29, 2025
Copilot AI review requested due to automatic review settings November 29, 2025 00:58
@ljunkie ljunkie force-pushed the rob/specific-casing branch from 865b4be to fdb9bc7 Compare November 29, 2025 00:59
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the brighterscript-formatter dependency from version 1.7.2025112720 to 1.7.2025112820 and adds new configuration options for more granular control over keyword and type casing in the BrightScript formatter. The update also includes a correction to an existing description that clarifies an example from "lower case" to "title case" to match the actual behavior.

  • Updates the brighterscript-formatter dependency version and changes the URL format from https://github.com/ to github: shorthand
  • Adds two new configuration options: specificKeywordCaseOverride and specificTypeCaseOverride for fine-grained casing control
  • Clarifies the keywordCaseOverride description to correctly state "title case" instead of "lower case"

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 5 comments.

File Description
package.json Updates brighterscript-formatter dependency URL format and version; adds two new formatter configuration options for specific keyword and type casing overrides; fixes description for keywordCaseOverride example
package-lock.json Updates the resolved version and commit hash for brighterscript-formatter dependency to match the new version
Comments suppressed due to low confidence (1)

package.json:61

  • [nitpick] Inconsistent dependency URL format. The brighterscript-formatter dependency now uses github:plexinc/brighterscript-formatter#... format while the brighterscript dependency on line 60 uses https://github.com/plexinc/brighterscript.git#... format. Consider using the same format for both dependencies for consistency.
        "brighterscript-formatter": "https://github.com/plexinc/brighterscript-formatter.git#1.7.2025112820",

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

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.


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

Copilot AI review requested due to automatic review settings November 29, 2025 01:09
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 2 changed files in this pull request and generated no new comments.


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

@ljunkie ljunkie merged commit c72fafd into xelp/main Nov 29, 2025
2 checks passed
@ljunkie ljunkie deleted the rob/specific-casing branch November 29, 2025 01:17
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