Skip to content

feat(template): add star request message to README#80

Open
fuleinist wants to merge 1 commit into
drkrillo:mainfrom
fuleinist:feat/add-star-request-message
Open

feat(template): add star request message to README#80
fuleinist wants to merge 1 commit into
drkrillo:mainfrom
fuleinist:feat/add-star-request-message

Conversation

@fuleinist
Copy link
Copy Markdown
Contributor

Summary

Adds a star request message above the issues table in the README template, addressing issue #71.

The message reads:

⭐ If this repo helped you find your first contribution, consider giving it a star — it really helps others discover it!

Changes

  • : Added the star request callout above the table

Context

Issue #71 notes that the repo has activity but visitors don't star it. This single-line callout is a low-friction way to remind people to star if the repo helped them.

Copy link
Copy Markdown
Owner

@drkrillo drkrillo left a comment

Choose a reason for hiding this comment

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

Hey @fuleinist !

This PR is perfectly executed, thanks for the contribution.

There is another earlier PR attempting to solve the same issue, but it still needs some modifications. Since it was opened first by a new contributor, and given the community nature of this project, I’d like to give them the opportunity to complete it.

You’ve already made a great contribution to the Python side of the project, and I think there are other open issues that could be a great fit if you’d like to keep collaborating.

Thanks again for the PR and for understanding. Please let me know if you have any questions.

@drkrillo
Copy link
Copy Markdown
Owner

I will keep this PR open in case the new contributor doesn't have a corrected version whitin a reasonable window time.

Copy link
Copy Markdown
Owner

@drkrillo drkrillo left a comment

Choose a reason for hiding this comment

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

Perfectly executed.

Please see my comment in the PR

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.

2 participants