Skip to content

Conversation

@j-c-c
Copy link
Collaborator

@j-c-c j-c-c commented Sep 23, 2024

Validation and clean up for the estimate_shifts method.

@j-c-c j-c-c added bug Something isn't working cleanup labels Sep 23, 2024
@j-c-c j-c-c self-assigned this Sep 23, 2024
@j-c-c j-c-c force-pushed the estimate_shifts branch 2 times, most recently from 49b29f6 to 1ac816e Compare September 30, 2024 18:44
garrettwrong
garrettwrong previously approved these changes Sep 30, 2024
@garrettwrong garrettwrong marked this pull request as ready for review September 30, 2024 18:51
Copy link
Collaborator

@garrettwrong garrettwrong left a comment

Choose a reason for hiding this comment

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

Nice job sifting through the shifts.

Thanks for fixing that last CI bug!

Copy link
Collaborator

@janden janden left a comment

Choose a reason for hiding this comment

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

Looks great!

@garrettwrong garrettwrong merged commit f3056c3 into develop Oct 7, 2024
33 checks passed
@garrettwrong garrettwrong deleted the estimate_shifts branch October 7, 2024 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working cleanup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants