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

Shorten sentence about defaults #2436

Merged
merged 2 commits into from
Feb 1, 2025

Conversation

jakirkham
Copy link
Member

PR Checklist:

  • note any issues closed by this PR with closing keywords
  • if you are adding a new page under docs/ or community/, you have added it to the sidebar in the corresponding _sidebar.json file
  • put any other relevant information below

@jakirkham jakirkham requested a review from a team as a code owner February 1, 2025 04:56
Copy link

netlify bot commented Feb 1, 2025

Deploy Preview for conda-forge-previews ready!

Name Link
🔨 Latest commit 8c4e927
🔍 Latest deploy log https://app.netlify.com/sites/conda-forge-previews/deploys/679e08b940d8ad0008078ff3
😎 Deploy Preview https://deploy-preview-2436--conda-forge-previews.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 62
Accessibility: 96
Best Practices: 100
SEO: 89
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@@ -22,7 +22,7 @@ You can [search](https://anaconda.org/) for packages online. Look out for packag

## Why conda-forge?

The packaging team from [Anaconda, Inc.](https://anaconda.org/), packages a multitude of packages and provides them to all users free of charge in their `defaults` channel.
The packaging team from [Anaconda, Inc.](https://anaconda.org/) provides a multitude of packages on their `defaults` channel.
Copy link
Member Author

Choose a reason for hiding this comment

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

This was likely written a long time ago. Given ToS, this really shouldn't say free of change to all users

However don't want to get into all of that in an intro. So just shortened this instead

Honestly this whole section may benefit from a revamp. In interim this seems like a small reasonable change

@beckermr
Copy link
Member

beckermr commented Feb 1, 2025

We need to fix the dockerhub links in another PR. :/

@beckermr beckermr enabled auto-merge (squash) February 1, 2025 11:42
@beckermr beckermr merged commit 7104f77 into conda-forge:main Feb 1, 2025
6 checks passed
@jakirkham
Copy link
Member Author

Thanks Matt! 🙏

Agreed. Yeah there are plenty of opportunities here to improve the docs

Think a lot of this was written in the mid to late 2010s. Would probably write different things now

For example we are explaining why conda-forge should exist from a standpoint of its creation. Well it does exist and it has so for a while

So maybe we would do better to talk about the communities we are serving

This is just one example 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants