Skip to content

Changes made to README.md#1690

Closed
vanshmehra wants to merge 2 commits into
conda-forge:mainfrom
vanshmehra:main
Closed

Changes made to README.md#1690
vanshmehra wants to merge 2 commits into
conda-forge:mainfrom
vanshmehra:main

Conversation

@vanshmehra

@vanshmehra vanshmehra commented Apr 5, 2022

Copy link
Copy Markdown

PR Checklist:

  • make all edits to the docs in the src directory, not in docs or in the html files
  • note any issues closed by this PR with closing keywords
  • put any other relevant information below

@vanshmehra vanshmehra requested a review from a team as a code owner April 5, 2022 14:35

@kathatherine kathatherine left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good! Just a few changes needed. Let me know if you have any questions.

Comment thread src/README.md Outdated
7. ``make html``


Conda-forge adheres to the NumFOCUS Code of Conduct. (https://numfocus.org/code-of-conduct)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This line has an unnecessary space at the very beginning.

Comment thread src/README.md Outdated

Conda-forge adheres to the NumFOCUS Code of Conduct. (https://numfocus.org/code-of-conduct)

## Scope

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Unnecessary space here, too. That's probably why the header isn't showing up as blue.

Comment thread src/README.md Outdated

## Scope

This code of conduct applies to all spaces managed by conda-forge, including all public and private mailing lists, issue trackers, wikis, forums, and any other communication channel used by our community. The code of conduct equally applies at conda-forge events and governs standards of behavior for attendees, speakers, volunteers, booth staff, and event sponsors.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Unnecessary space here, too.

Comment thread src/README.md Outdated
- Inappropriate physical contact and/or unwelcome sexual attention or sexual comments
- Sexist, racist, or otherwise discriminatory jokes and language
- Trolling or insulting and derogatory comments
- Written or verbal comments which have the effect of excluding people on the basis of membership in a specific group, including level of experience, gender, gender identity and expression, sexual orientation, disability, neurotype, personal appearance, body size, race, ethnicity, age, religion, or nationality

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

There's a double space between "of" and "experience that doesn't need to be there.

Comment thread src/README.md

This code of conduct applies to all spaces managed by conda-forge, including all public and private mailing lists, issue trackers, wikis, forums, and any other communication channel used by our community. The code of conduct equally applies at conda-forge events and governs standards of behavior for attendees, speakers, volunteers, booth staff, and event sponsors.


Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

There should only be one blank line in-between sections in this document, instead of two or more, so I'd recommend going through and taking those extra blank lines out.

@vanshmehra

Copy link
Copy Markdown
Author

I have made the changes as requested. Since this is my first time contributing to open source, I apologise for all the silly mistakes I made. Thank you for your time :)

@kathatherine kathatherine left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks so much! And don't worry about mistakes. This is about you learning and sometimes we learn by doing something wrong first. That's what these reviews are for.

@jaimergp

Copy link
Copy Markdown
Member

Superseded by #1640

@jaimergp jaimergp closed this Jun 22, 2023
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