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

Added new FAQs and resolved 'content' labeled FAQs #137

Merged
merged 34 commits into from
Jun 8, 2023

Conversation

Pinak-Datta
Copy link
Contributor

@Pinak-Datta Pinak-Datta commented May 17, 2023

Description

This PR resolves the following:

Initial problem: The FAQ section was lacking certain questions and answers, and there were unresolved FAQs labeled as 'content' on GitHub.

Choices to be aware of:

  • Ensure the new FAQs are comprehensive and cover common user queries.
  • Verify that the resolved FAQs accurately address the issues mentioned in the 'content' labeled FAQs.
  • Review and test the changes thoroughly before merging the PR.

@netlify
Copy link

netlify bot commented May 17, 2023

Deploy Preview for conda-dot-org ready!

Name Link
🔨 Latest commit 7f56998
🔍 Latest deploy log https://app.netlify.com/sites/conda-dot-org/deploys/6481d13ac96da3000878792e
😎 Deploy Preview https://deploy-preview-137--conda-dot-org.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

Copy link
Contributor

@kathatherine kathatherine left a comment

Choose a reason for hiding this comment

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

Lots of great stuff here, Pinak. Thanks for getting going on this so quickly!

I'm sure everyone else will have opinions on how we're approaching talking about conda here. I've left you some questions and comments to consider. @travishathaway, @beeankha, @jezdez, @jaimergp, @tnabtaf, would also love your input, if you have time.

learn/faq.md Outdated Show resolved Hide resolved
learn/faq.md Outdated Show resolved Hide resolved
learn/faq.md Outdated Show resolved Hide resolved
learn/faq.md Outdated
Comment on lines 29 to 36
To install a non-Python package, you can use the following command:
```
conda install -c conda-forge <package-name>
```
For example, to install a non-Python package like gcc for C/C++ programming language, you can run:
```
conda install -c conda-forge gcc
```
Copy link
Contributor

Choose a reason for hiding this comment

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

@jaimergp, I'd love opinions on this.

You can get non-Python packages from more than conda-forge and specifying conda-forge here makes it seem like it's the only option, so we might want to explain that.

@@ -18,6 +51,33 @@ For more information, please check out the [conda documentation page on channels

[Conda-forge](https://conda-forge.org/) is a [community-led GitHub organization](https://github.com/conda-forge) that provides access to thousands of conda package recipes. All of these recipes are open source and can be installed with the conda package manager by specifying conda-forge as the channel.
Copy link
Contributor

Choose a reason for hiding this comment

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

I know this isn't your addition, Pinak, but we should be sure we're consistent with the conda-forge naming.

@jaimergp, is it supposed to always be lowercase, or is capitalizing at the beginning of a sentence fine?

learn/faq.md Show resolved Hide resolved
learn/faq.md Outdated Show resolved Hide resolved
learn/faq.md Outdated Show resolved Hide resolved
learn/faq.md Outdated Show resolved Hide resolved
learn/faq.md Outdated Show resolved Hide resolved
learn/faq.md Outdated Show resolved Hide resolved
learn/faq.md Outdated Show resolved Hide resolved
learn/faq.md Outdated Show resolved Hide resolved
learn/faq.md Outdated Show resolved Hide resolved
learn/faq.md Outdated Show resolved Hide resolved
learn/faq.md Outdated Show resolved Hide resolved
learn/faq.md Outdated Show resolved Hide resolved
learn/faq.md Outdated Show resolved Hide resolved
learn/faq.md Outdated Show resolved Hide resolved
learn/faq.md Outdated Show resolved Hide resolved
Pinak-Datta and others added 22 commits May 31, 2023 12:14
Implemented Conda capitalization

Co-authored-by: Bianca Henderson <[email protected]>
Implemented Conda capitalization

Co-authored-by: Bianca Henderson <[email protected]>
Implemented Conda capitalization

Co-authored-by: Bianca Henderson <[email protected]>
Implemented proper capitalization formats

Co-authored-by: Bianca Henderson <[email protected]>
using "on" instead of "in"

Co-authored-by: Bianca Henderson <[email protected]>
Implemented Conda capitalization

Co-authored-by: Bianca Henderson <[email protected]>
Implemented necessary suggestions, replacing "the" with "then".

Co-authored-by: Bianca Henderson <[email protected]>
Implemented Conda capitalization

Co-authored-by: Bianca Henderson <[email protected]>
Implemented Conda capitalization

Co-authored-by: Bianca Henderson <[email protected]>
Implemented Conda capitalization

Co-authored-by: Bianca Henderson <[email protected]>
Replacing "you" with "The person building..."

Co-authored-by: Bianca Henderson <[email protected]>
Implemented proper Conda Capitalization, and "your" replaces "the"

Co-authored-by: Bianca Henderson <[email protected]>
Implemented Conda Capitalization

Co-authored-by: Bianca Henderson <[email protected]>
Implemented Conda Capitalization

Co-authored-by: Bianca Henderson <[email protected]>
Implemented Conda Capitalization

Co-authored-by: Bianca Henderson <[email protected]>
Implemented Conda Capitalization rule

Co-authored-by: Bianca Henderson <[email protected]>
Implemented Conda Capitalization rule

Co-authored-by: Bianca Henderson <[email protected]>
Implemented Conda Capitalization rule

Co-authored-by: Bianca Henderson <[email protected]>
Implemented Conda Capitalization rule

Co-authored-by: Bianca Henderson <[email protected]>
Implemented Conda Capitalization rule

Co-authored-by: Bianca Henderson <[email protected]>
Implemented Conda Capitalization rule where Conda can be capitalized when starting a new sentence.

Co-authored-by: Bianca Henderson <[email protected]>
Implemented Conda Capitalization rule where Conda can be capitalized when starting a new sentence.

Co-authored-by: Bianca Henderson <[email protected]>
@Pinak-Datta Pinak-Datta requested a review from beeankha May 31, 2023 07:00
All feedback like Conda capitalizations, accurate description of communities, not just limiting the answer to conda, and many more that were suggested initially.
learn/faq.md Outdated Show resolved Hide resolved
learn/faq.md Show resolved Hide resolved
learn/faq.md Show resolved Hide resolved
learn/faq.md Outdated Show resolved Hide resolved
learn/faq.md Outdated Show resolved Hide resolved
learn/faq.md Show resolved Hide resolved
learn/faq.md Outdated Show resolved Hide resolved
learn/faq.md Show resolved Hide resolved
learn/faq.md Outdated Show resolved Hide resolved
learn/faq.md Outdated Show resolved Hide resolved
Copy link
Contributor

@kathatherine kathatherine left a comment

Choose a reason for hiding this comment

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

Thanks for all your work on this! I applied some formatting changes real quick. You'll want to git pull in those changes if you make any more edits on your fork. I want to remind people that this exists at the conda.org community meeting on Wednesday and give them a chance to look it over before we merge in.

@kathatherine kathatherine self-requested a review June 3, 2023 17:28
@Pinak-Datta
Copy link
Contributor Author

Thanks for all your work on this! I applied some formatting changes real quick. You'll want to git pull in those changes if you make any more edits on your fork. I want to remind people that this exists at the conda.org community meeting on Wednesday and give them a chance to look it over before we merge in.

Ok, Thanks!

@travishathaway
Copy link
Contributor

I noticed this branch was out of sync with main in some weird ways. I made a lot of manual updates to make sure everything would be okay after the merge.

Copy link
Contributor

@travishathaway travishathaway left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@kathatherine kathatherine merged commit 97bdb02 into conda:main Jun 8, 2023
@kathatherine
Copy link
Contributor

@Pinak-Datta Congrats on your first merge in the repo! If you could list the GitHub tickets that this PR closes by number in the description at the top of this PR, that'd be great. That way we can make sure any open tickets are closed properly.

@Pinak-Datta
Copy link
Contributor Author

Pinak-Datta commented Jun 8, 2023

@Pinak-Datta Congrats on your first merge in the repo! If you could list the GitHub tickets that this PR closes by number in the description at the top of this PR, that'd be great. That way we can make sure any open tickets are closed properly.

Done adding the list of tickets that this PR Closes.

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