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

Fix conda badge in README #17345

Merged
merged 4 commits into from
Apr 27, 2023
Merged

Fix conda badge in README #17345

merged 4 commits into from
Apr 27, 2023

Conversation

asmith26
Copy link
Contributor

@asmith26 asmith26 commented Apr 12, 2023

Following on from #17289 it would appear that the official conda-forge package is https://anaconda.org/conda-forge/lightning

Conda

However, the conda badge in the README points to https://anaconda.org/conda-forge/pytorch-lightning

Assuming I've understood correctly that lightning is the official conda-forge package, would it also be worth adding a deprecation warning to pytorch-lightning (and/or redirect to lightning)?

Thanks! :)

Following on from #17289 it would appear that the official conda-forge package is https://anaconda.org/conda-forge/lightning

However, the conda badge in the [README](https://github.com/Lightning-AI/lightning/edit/master/README.md) points to https://anaconda.org/conda-forge/pytorch-lightning

Assuming I've understood correctly that [`lightning`](https://anaconda.org/conda-forge/lightning) is the official conda-forge package, would it also be worth adding a deprecation warning to [`pytorch-lightning`](https://anaconda.org/conda-forge/pytorch-lightning) (and/or redirect to [`lightning`](https://anaconda.org/conda-forge/lightning))?

Thanks for any help! :)
@asmith26 asmith26 changed the title Fix conda badge Fix conda badge in README Apr 12, 2023
@Borda Borda added the docs Documentation related label Apr 14, 2023
@Borda Borda enabled auto-merge (squash) April 18, 2023 11:03
@Borda Borda merged commit 8c18439 into Lightning-AI:master Apr 27, 2023
@mergify mergify bot added the ready PRs ready to be merged label Apr 27, 2023
Borda pushed a commit that referenced this pull request May 9, 2023
(cherry picked from commit 8c18439)
Borda pushed a commit that referenced this pull request Jun 1, 2023
(cherry picked from commit 8c18439)
lantiga pushed a commit that referenced this pull request Jun 2, 2023
(cherry picked from commit 8c18439)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation related ready PRs ready to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants