Skip to content

Conversation

@cypriansakwa
Copy link
Contributor

@cypriansakwa cypriansakwa commented Jul 28, 2025

Hi @critesjosh

Description

This PR updates the nightly continuous integration (CI) workflows to automatically upgrade npm package dependencies to their latest published versions before running tests. This helps detect breaking changes from upstream packages early and ensures our test suite is always validating against the newest dependency versions.

Problem*

Resolves noir-lang/noir-examples/issues/86

Summary*

Nightly CI workflows now upgrade npm package dependencies to the latest published versions before running tests, ensuring compatibility with upstream changes and improving reliability. This is done using npm-check-updates (ncu -u) in the nightly scheduled jobs for all relevant frontend projects.

Additional Context

This change helps detect breaking changes in dependencies early by running the latest package versions in our test suites. It affects the nightly CI workflows for projects such as recursion/js, solidity-example/js, and web-starter/web/*.

PR Checklist*

  • I have tested the changes locally.
  • I have formatted the changes with Prettier and/or cargo fmt on default settings.

@cypriansakwa cypriansakwa marked this pull request as draft July 28, 2025 22:29
@cypriansakwa cypriansakwa marked this pull request as ready for review July 29, 2025 14:27
@cypriansakwa
Copy link
Contributor Author

Hi @critesjosh .
I noticed a merge conflict in the workflow file related to bb installation. I have resolved the conflict. If you’d like me to adjust anything, I will be happy to make changes. Thanks for your patience.

@cypriansakwa
Copy link
Contributor Author

Hi @critesjosh and @signorecello 👋,

Just a gentle reminder to please take a moment to review this PR when you get a chance.
Your feedback will be really helpful in moving it forward. 🙏

Thanks a lot for your time and support!

@cypriansakwa
Copy link
Contributor Author

Hi @critesjosh.
I’ve moved the commands into the “Install bbup” step for consistency with the other workflows.
Thanks.

@cypriansakwa
Copy link
Contributor Author

Hi @critesjosh and @signorecello 👋,

Just a gentle reminder to please take a moment to review this PR when you get a chance.
Your feedback will be really helpful in moving it forward. 🙏

Thanks a lot for your time and support!

@cypriansakwa
Copy link
Contributor Author

Hi @critesjosh and @signorecello 👋,

Just a gentle reminder to please take a moment to review this PR when you get a chance.
Your feedback will be really helpful in moving it forward. 🙏

Thanks a lot for your time and support!

1 similar comment
@cypriansakwa
Copy link
Contributor Author

Hi @critesjosh and @signorecello 👋,

Just a gentle reminder to please take a moment to review this PR when you get a chance.
Your feedback will be really helpful in moving it forward. 🙏

Thanks a lot for your time and support!

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.

Update the nightly tests to run that latest published version of npm packages

2 participants