Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(#9768): uplift sinon to 19+ #9780

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

akshat16j
Copy link

@akshat16j akshat16j commented Feb 5, 2025

Uplifted sinon to 19+ and modified the useFakeTimes function.

Closes #9768

Code review checklist

  • Readable: Concise, well named, follows the style guide, documented if necessary.
  • Documented: Configuration and user documentation on cht-docs
  • Tested: Unit and/or e2e where appropriate
  • Internationalised: All user facing text
  • Backwards compatible: Works with existing data and configuration or includes a migration. Any breaking changes documented in the release notes.

License

The software is provided under AGPL-3.0. Contributions to this project are accepted under the same license.

Uplifted sinon to 19+ and modified the useFakeTimes function.
@jkuester jkuester self-requested a review February 5, 2025 16:36
@jkuester jkuester changed the title #9768 Uplifted sinon to 19+ chore(#9768): uplift sinon to 19+ Feb 5, 2025
Copy link
Contributor

@jkuester jkuester left a comment

Choose a reason for hiding this comment

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

@akshat16j Thanks for the PR! Looks like the linting is failing, but if you can get that fixed and the build passes we should be able to get this merged!

FYI, you can run the linting locally via the npm run lint command. 👍

@akshat16j
Copy link
Author

Ok! will definitely do that.

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.

Uplift sinon to 19+
2 participants