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

docs: Add conda-forge feedstock to install instructions #310

Merged

Conversation

matthewfeickert
Copy link
Contributor

@matthewfeickert matthewfeickert commented Aug 7, 2024

Following my excitement of realizing that there already was a conda-forge feedstock (https://github.com/conda-forge/nvtop-feedstock) in #76 (comment), this PR notes the existence of the conda-forge feedstock in the README and adds install instructions for the conda family of package managers as well as for pixi.

* Note the existence of the conda-forge feedstock in the README and add
  install instructions for the conda family of package managers as well
  as for pixi.
   - c.f. https://github.com/conda-forge/nvtop-feedstock
@@ -39,6 39,7 @@ Table of Contents
- [Arch Linux](#arch-linux)
- [AppImage](#appimage)
- [Snap](#snap)
- [Conda-forge](#conda-forge)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I added conda-forge at the end of the list of package manager options, but before Linux containers, to avoid having to make position selection choices, but if this should get moved to a different spot I'm happy to do so.

@matthewfeickert
Copy link
Contributor Author

@Syllo This should be ready for review now.

@Syllo
Copy link
Owner

Syllo commented Aug 31, 2024

Looks good,
Thank you for your contribution.

@Syllo Syllo merged commit a2faf1f into Syllo:master Aug 31, 2024
@matthewfeickert matthewfeickert deleted the docs/add-conda-forge-information branch September 1, 2024 17: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.

2 participants