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.

Sign up for GitHub

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

Jump to bottom

fix!: Replace Time.now with Process.clock_gettime #717

Merged
merged 8 commits into from Apr 28, 2021
Merged

fix!: Replace Time.now with Process.clock_gettime #717

merged 8 commits into from Apr 28, 2021

Conversation

fbogsany
Copy link
Contributor

@fbogsany fbogsany commented Apr 20, 2021 •
edited

Fixes #709

robertlaurin
robertlaurin previously approved these changes Apr 20, 2021
View reviewed changes
@robertlaurin robertlaurin self-requested a review April 20, 2021 14:32
@fbogsany fbogsany changed the title fix!: Replace Time.now with Process.clock_gettime WIP fix!: Replace Time.now with Process.clock_gettime Apr 20, 2021
@fbogsany fbogsany mentioned this pull request Apr 20, 2021
Closed
@fbogsany fbogsany changed the title WIP fix!: Replace Time.now with Process.clock_gettime fix!: Replace Time.now with Process.clock_gettime Apr 28, 2021
robertlaurin
robertlaurin approved these changes Apr 28, 2021
View reviewed changes
Copy link
Contributor

@robertlaurin robertlaurin left a comment

Choose a reason for hiding this comment

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

🕐

@fbogsany fbogsany merged commit 50ce54b into main Apr 28, 2021
@fbogsany fbogsany deleted the process-clock-gettime branch April 28, 2021 20:55
@fbogsany fbogsany mentioned this pull request May 29, 2021
Merged
2 tasks
@kaylareopelle kaylareopelle mentioned this pull request Aug 22, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@robertlaurin robertlaurin robertlaurin approved these changes

@dazuma dazuma Awaiting requested review from dazuma dazuma is a code owner

@ericmustin ericmustin Awaiting requested review from ericmustin ericmustin is a code owner

@mwear mwear Awaiting requested review from mwear mwear is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Replace use of Time.now with Process.clock_gettime
2 participants
@fbogsany @robertlaurin