Skip to content

Conversation

@IAlibay
Copy link
Member

@IAlibay IAlibay commented May 13, 2023

Currently up to analysis step

@github-actions
Copy link

Binder 👈 Launch a binder notebook on branch OpenFreeEnergy/ExampleNotebooks/update-cli-tutorial-IA

@IAlibay IAlibay changed the title Update cli_tutorial.md Updat cli_tutorial.md May 13, 2023
@IAlibay IAlibay changed the title Updat cli_tutorial.md Fix typos in cli_tutorial.md May 13, 2023
@IAlibay IAlibay requested a review from dwhswenson May 13, 2023 17:49
@IAlibay
Copy link
Member Author

IAlibay commented May 13, 2023

That's all my comments for the CLI portion, I'll look at the notebook next.

Copy link
Member

@dwhswenson dwhswenson left a comment

Choose a reason for hiding this comment

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

Minor fixes to the fixes.

@dwhswenson
Copy link
Member

This... probably should have been merged BEFORE the workshop? 🤷‍♂️ Merging now.

@dwhswenson dwhswenson merged commit e54e6b8 into master May 25, 2023
@dwhswenson dwhswenson deleted the update-cli-tutorial-IA branch May 25, 2023 18:12
dwhswenson added a commit that referenced this pull request May 25, 2023
* Update cli_tutorial.md

* more typos

* Update rbfe_tutorial/cli_tutorial.md

* Update rbfe_tutorial/cli_tutorial.md

---------

Co-authored-by: Irfan Alibay <IAlibay@users.noreply.github.com>
@IAlibay
Copy link
Member Author

IAlibay commented May 25, 2023

🤦🏽

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