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

Chronos setTestNow for apps test harness default #984

Merged
merged 1 commit into from
Dec 1, 2023

Conversation

dereuromark
Copy link
Member

Shouldnt we include that as best practice in general?

@dereuromark dereuromark added this to the 5.x milestone Nov 29, 2023
@dereuromark dereuromark changed the title Chronos fixNow for apps Chronos setTestNow for apps test harness default Nov 30, 2023
@markstory markstory merged commit cf324d7 into 5.x Dec 1, 2023
4 checks passed
@markstory markstory deleted the dereuromark-patch-1 branch December 1, 2023 02:40
@markstory
Copy link
Member

Shouldnt we include that as best practice in general?

I agree, I've had to do this in a few times.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants