Skip to content

Upgrade axios and lodash versions#110

Merged
bwang-icf merged 1 commit into
mainfrom
ryanmorosa/bb2-4755-update-lodash-and-axios
May 11, 2026
Merged

Upgrade axios and lodash versions#110
bwang-icf merged 1 commit into
mainfrom
ryanmorosa/bb2-4755-update-lodash-and-axios

Conversation

@ryan-morosa
Copy link
Copy Markdown
Contributor

JIRA Ticket:
BB2-4755

What Does This PR Do?

  • Upgrade lodash to version 4.18.1 or higher.
  • Upgrade axios to version 1.15.2 or higher.

What Should Reviewers Watch For?

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

Validation

  • Run npm ls axios and yarn list axios, confirm you see 1.16.0
  • Run npm ls lodash and yarn list lodash, confirm you see 4.18.1
  • Run yarn test or npm test, confirm all tests pass. Ensure yarn build or npm build runs successfully, but you may encounter issues with yarn build:types (will be a follow-on story).
    • If having issues with any of these, make sure to remove node_modules and yarn.lock and ensure you have .bluebutton-config.json in src folder.

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.

@sb-DarenDean
Copy link
Copy Markdown

All good here as well. Have another engineer look over and approve. Thanks.

Copy link
Copy Markdown
Contributor

@bwang-icf bwang-icf left a comment

Choose a reason for hiding this comment

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

verified with local test

@bwang-icf bwang-icf merged commit 6549491 into main May 11, 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.

3 participants