Skip to content

Update axios and ncc dependencies in package.json#9

Merged
bkundu929 merged 1 commit into
masterfrom
update/axios
May 7, 2026
Merged

Update axios and ncc dependencies in package.json#9
bkundu929 merged 1 commit into
masterfrom
update/axios

Conversation

@bkundu929
Copy link
Copy Markdown
Contributor

Copy link
Copy Markdown
Contributor Author

@bkundu929 bkundu929 left a comment

Choose a reason for hiding this comment

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

Notes for review.

Additionally, and for future reference, the QA testing for this will be performed in two rounds:

  • While the PR is still a draft, QA will first use the commit SHA to reference the updates in the main.yml file. They’ll also update the secrets etc as required, and run the workflow to see if the post-update behavior is fine. At this point, dev needs to work with QA to run or stop the GitHub Action runner as and when required, update secrets and other configuration info.
  • If fine, then mark the PR as ready for review. If code review goes fine, merge the PR, and this will publish the new bumped version of the package.json. At this point, QA can use the new version tag in the main.yml file instead of the commit hash, and smoke test to make sure everything works fine again.

Comment thread package.json
Comment thread package.json
@bkundu929 bkundu929 marked this pull request as ready for review May 6, 2026 15:58
@bkundu929 bkundu929 requested a review from tylercamp May 6, 2026 15:58
@bkundu929 bkundu929 merged commit d0a0a9c into master May 7, 2026
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