- Merge request !43Issue #3330770: Replace README.txt with README.md and update โ (Open) created by shubham rathore
- ๐ฎ๐ณIndia Sonal Gyanani
Thanks, MR looks good to me so moving it to RTBC
- Assigned to Kristen Pol
- ๐บ๐ธUnited States Kristen Pol Santa Cruz, CA, USA
Assigning to myself as I'm triaging all RTBC issues.
- Issue was unassigned.
- Status changed to Closed: won't fix
over 1 year ago 4:11am 9 August 2023 - ๐บ๐ธUnited States Kristen Pol Santa Cruz, CA, USA
I've checked the issue credits for the users above and have found that most/all of them have made a practice of creating these
README.txt
toREADME.md
issues in what appears to be an attempt to game the issue credit system.Please see: ๐ฑ Categorize contributions that could be considered 'gaming the credit system' and propose solutions (policy, automation, etc) Active
Although there may be some merit in this issue as it shows on GitLab, I am closing as "won't fix" to dissuade this type of contribution. We may separately update the README file as maintainers but not as part of this issue.
- Status changed to Needs review
over 1 year ago 9:32am 9 August 2023 - ๐ฉ๐ฐDenmark ressa Copenhagen
Thanks for looking at this @Kristen Pol.
I just posted a more general comment ๐ฑ Categorize contributions that could be considered 'gaming the credit system' and propose solutions (policy, automation, etc) Active about how I see the challenges of preventing gaming of the credit system in the issue you link to.
I do think updating the format to Markdown would be an improvement, which shouldn't automatically be categorized as credit gaming, and commented:
It has been discussed at length, and decided that the Markdown works better than txt in #3192842: Make our README more welcoming by converting it into an "entrypoint" into the Drupal ecosystem โ , which I very much agree with. By now, txt README's look like relics from the 90's to me, since most projects now use Markdown on Github and Gitlab.
Now, about this issue, which triggered my comment ...
What we could do is evaluate the individual contributions from the users above who committed changes, and grant credit to those who added useful changes?
I just went through the changes, and from what I can tell, it's mostly the last commit, you are aiming your critique at, and I agree that the update by @Manoj Raj.R looks redundant.
But I actually think linking to Pathauto and Pathologic under "Recommended modules" is a great idea, and I would grant credit to @Shubham Rathore and @fanton.
- ๐บ๐ธUnited States Kristen Pol Santa Cruz, CA, USA
Thank you @ressa for chiming in here. I will certainly reconsider my position based on your feedback. I might not be able to look at this again properly until tomorrow but I did want you to know that I saw this and appreciate your candid input.
- ๐ฉ๐ฐDenmark ressa Copenhagen
Thank you @Kristen Pol for a fast response, and for you willingness to take a second look at this. And I do feel a bit bad, since you are a very considerate and friendly presence in the issue queues, so please don't take this personal. And there's no pressure for a fast review, it can easily wait until you have time.
- ๐บ๐ธUnited States Kristen Pol Santa Cruz, CA, USA
Please donโt feel bad at all! You are giving valid and needed feedback on a delicate subject and I very much appreciate the input. I am happy you felt comfortable providing this feedback. I want to be in a community that is open to such candid discussions.
- ๐ฎ๐ณIndia vishal prasad
Vishal Prasad โ made their first commit to this issueโs fork.
- last update
about 1 year ago 63 pass - @vishal-prasad opened merge request.
- ๐ฎ๐ณIndia vishal prasad
Hi i just made the changes to the fork, and modified the
Readme.txt
to<h2>Readme.md</h2>
. - last update
about 1 year ago 63 pass - last update
about 1 year ago 63 pass - last update
about 1 year ago 63 pass - last update
about 1 year ago 63 pass - last update
about 1 year ago 63 pass - Status changed to Needs work
6 months ago 6:43am 7 June 2024 - ๐ฎ๐ณIndia deepak5423
@Vishal Prasad !MR55 is not following the README.md standards. Please follow the README.md โ standards.
- ๐ฎ๐ณIndia deepak5423
deepak5423 โ changed the visibility of the branch 8.x-1.x to hidden.
- Status changed to Needs review
6 months ago 10:28am 10 June 2024 - Status changed to RTBC
5 months ago 11:17am 13 June 2024 - ๐ฎ๐ณIndia kalash-j jaipur
@deepak5423 !MR43 is following all the README.md standards.