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

Refactor/update operation syntax #56

Merged
merged 14 commits into from
Nov 15, 2023
Merged

Conversation

b-butler
Copy link
Member

Updates the examples to the current flow API.

@b-butler b-butler requested review from a team as code owners October 31, 2022 16:37
@b-butler b-butler requested review from tcmoore3 and shihkual and removed request for a team October 31, 2022 16:37
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@bdice
Copy link
Member

bdice commented Oct 31, 2022

CI is failing -- @b-butler let me know when this is ready to review (or if help is needed).

@b-butler
Copy link
Member Author

@bdice This is failing because the flow PR has yet to be approved or merged.

@bdice
Copy link
Member

bdice commented Nov 1, 2022

@bdice This is failing because the flow PR has yet to be approved or merged.

Whoops. Somehow I thought it had been merged already. I approved glotzerlab/signac-flow#681.

Use a more appropriate verb.

Co-authored-by: Bradley Dice <[email protected]>
@b-butler b-butler force-pushed the refactor/update-operation-syntax branch from aadbab9 to 4c19bf7 Compare December 9, 2022 19:40
Copy link
Member

@tcmoore3 tcmoore3 left a comment

Choose a reason for hiding this comment

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

Looks good, but CI still failing.

@bdice
Copy link
Member

bdice commented Nov 11, 2023

@b-butler Is this PR still needed? The CI has been updated so we should be able to fix it up and merge, or close if not.

@b-butler b-butler force-pushed the refactor/update-operation-syntax branch from e6cee18 to cb0cf3f Compare November 13, 2023 22:59
@bdice
Copy link
Member

bdice commented Nov 14, 2023

@b-butler I wasn't sure if this is ready to review/merge but the current changes look fine to me. Please merge if you're ready.

@b-butler b-butler force-pushed the refactor/update-operation-syntax branch from 7feb2b5 to 689fb6f Compare November 14, 2023 19:55
@b-butler b-butler merged commit 119029e into main Nov 15, 2023
3 checks passed
@b-butler b-butler deleted the refactor/update-operation-syntax branch November 15, 2023 18:59
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.

3 participants