Skip to content

Conversation

parthamk
Copy link

@parthamk parthamk commented Jan 18, 2023

What:

Why:

How:

Checklist:

  • Documentation
  • Ready to be merged
  • Added myself to contributors table.
    Bot Usage

@parthamk parthamk requested a review from a team as a code owner January 18, 2023 12:01
@parthamk
Copy link
Author

I want the readme should be more fun to read and precise, if you like it please accept the pull request. Thanks.

Copy link

@a3fckx a3fckx left a comment

Choose a reason for hiding this comment

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

The initial Paragraph and last paragraph looks fine but the main content can be listed down in bullets.

or because they do not feel like it is significant enough. In these cases, you should still encourage them to add themselves,
but it may be necessary for you to add the contributor yourself (though it's a good idea to do so in the form of a pull
request to make sure they're ok with being added).
As the captain of the ship or the ringleader of the repository, it's important to remember that keeping the list of contributors shipshape and Bristol fashion shouldn't fall solely on your shoulders. Instead, it's crucial to actively rally and empower others to keep the list current and polished.
Copy link
Member

Choose a reason for hiding this comment

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

@parthamk this is a lovely addition to this page - it's making me smile reading it. ✨ This PR is SOOOO old, but I'd love to merge it. We are reworking the documentation to a new platform, and the path is now src/docs/repository-maintainers.md

If others here are ok with the lightness of this pr - i wonder if we can modify it to go to the new directory structure in our docs-upgrade branch (rather than main/master)

@lwasser
Copy link
Member

lwasser commented Sep 9, 2025

@all-contributors please add @parthamk for docs

Copy link
Contributor

@lwasser

I've put up a pull request to add @parthamk! 🎉

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