Skip to content

Elaborate on managing Conda python, channels #606

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

Merged
merged 10 commits into from
Mar 20, 2024
Merged

Conversation

muyuxuan946
Copy link
Collaborator

@muyuxuan946 muyuxuan946 commented Mar 20, 2024

This PR solves the aspects mentioned in Issue #484: Development Process: Introduction for the Conda #484. It added instructions for five major types of Conda tasks including Managing Python, Managing virtual packages, Creating custom channels, Creating projects, and Viewing command-line help. The instruction introduces the commonly used commands for these tasks.

@muyuxuan946 muyuxuan946 requested a review from emma0925 March 20, 2024 01:48
@muyuxuan946 muyuxuan946 changed the title creating new a2 myx conda Elaborate on managing Conda python, channels Mar 20, 2024
Copy link
Collaborator

@emma0925 emma0925 left a comment

Choose a reason for hiding this comment

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

Hi Yuxuan,
Your writing is very detail and clear!
One suggestion I have is to be consistent with the file structure of pr #604, I would suggest you to break each sub-section into different files.
Also, please resolve the conflict with setup_conda.md in the main branch. I think by break all the sections down to different files. The conflict will be resolved.

@emma0925
Copy link
Collaborator

emma0925 commented Mar 20, 2024

Hi Yuxuan, I fogot to add citations for my pr #600, it's already merged. Could you add the citation section for me in the setup_conda.md, plz.

References

@muyuxuan946
Copy link
Collaborator Author

will do that right now. Thank you for feedback.

@YankeMao
Copy link
Collaborator

I'd suggest to edit the reference list to keep consist with the other documents for Conda in the repo

@YankeMao
Copy link
Collaborator

Also, the reference list is needed for each .md file for the consistent aim.

Copy link
Collaborator

@YankeMao YankeMao left a comment

Choose a reason for hiding this comment

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

Good job! Approved.

Copy link
Collaborator

@emma0925 emma0925 left a comment

Choose a reason for hiding this comment

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

LGTM

@muyuxuan946 muyuxuan946 merged commit 5837137 into main Mar 20, 2024
@muyuxuan946 muyuxuan946 deleted the CSC301_winter_myx branch March 20, 2024 03:39
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