-
Notifications
You must be signed in to change notification settings - Fork 35
make CLI starting guide easier to find from the landing page #1787
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
base: main
Are you sure you want to change the base?
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1787 +/- ##
==========================================
- Coverage 95.54% 93.18% -2.37%
==========================================
Files 197 197
Lines 17046 17046
==========================================
- Hits 16287 15884 -403
- Misses 759 1162 +403
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
blocked by #1792, I want to check that everything renders properly in the preview before merging. |
Pull request was converted to draft
|
No API break detected ✅ |
We should point users to the tutorial as the starting point, rather than the CLI api.
Blocked by OpenFreeEnergy/ExampleNotebooks#283
Checklist
newsentry, or the changes are not user-facing.pre-commit.ci autofix.Manual Tests: these are slow so don't need to be run every commit, only before merging and when relevant changes are made (generally at reviewer-discretion).
Developers certificate of origin