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

Adding y to install #207

Merged
merged 8 commits into from
Jun 4, 2024
Merged

Conversation

shimwell
Copy link
Collaborator

@shimwell shimwell commented Jun 4, 2024

Description

Just worked through the install with @AlvaroCubi and he spotted some ideas for making the install a bit easier. This PR adds -y to the conda and mamba install instructions which saves the user having to enter Y to proceed

Fixes issue

Checklist

  • I'm making a PR from a feature branch on my fork into GEOUNED-org/GEOUNED/dev branch
  • I have followed PEP8 style guide for Python or run a formatter such as black or ruff format on my code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works

@shimwell
Copy link
Collaborator Author

shimwell commented Jun 4, 2024

this one is ready for review

@psauvan psauvan merged commit 58c245e into GEOUNED-org:dev Jun 4, 2024
3 checks passed
psauvan pushed a commit that referenced this pull request Jun 17, 2024
* moved two methods out of start

* removed unnecessary logger line

* changes while working therough the install
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.

2 participants