Skip to content

BB2-4853: Update fast-uri#116

Merged
JamesDemeryNava merged 2 commits into
mainfrom
jamesdemery/bb2-4853-fast-uri-update
Jun 16, 2026
Merged

BB2-4853: Update fast-uri#116
JamesDemeryNava merged 2 commits into
mainfrom
jamesdemery/bb2-4853-fast-uri-update

Conversation

@JamesDemeryNava

Copy link
Copy Markdown
Contributor

JIRA Ticket:
BB2-4853

What Does This PR Do?

Updates fast-uri to 3.1.2

What Should Reviewers Watch For?

If you're reviewing this PR, please check for these things in particular:

Validation

  • Run npm ls fast-uri and yarn list fast-uri, confirm you see 3.1.2
  • Confirm you can run yarn build successfully
    • Note, there are issues with yarn build:types and yarn test. Those commands will not work, and were broken before this ticket.

What Security Implications Does This PR Have?

Please indicate if this PR does any of the following:

  • Adds any new software dependencies
  • Modifies any security controls
  • Adds new transmission or storage of data
  • Any other changes that could possibly affect security?
  • Yes, one or more of the above security implications apply. This PR must not be merged without the ISSO or team security engineer's approval.

@bwang-icf bwang-icf 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.

I see that npm ls fast-uri return 3.1.2, but yarn does not. I believe this is because I don't have a yarn.lock locally, but since we want to also move to just one package manager and the npm package manager is preferred and works, I am good to approve this.

@JamesDemeryNava JamesDemeryNava merged commit bc2820a into main Jun 16, 2026
4 checks passed
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.

4 participants