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

Replace "prior to" with "immediately after" in bundle gem generated README file #6338

Merged

Conversation

sambostock
Copy link
Contributor

What was the end-user or developer problem that led to this PR?

Currently, the instructions and placeholder contradict each other.

The commit that introduced this placeholder (42bc471) makes it clear that the placeholder should be replaced AFTER publishing (i.e. once the author successfully claims the gem name on Rubygems), so the placeholder should match.

What is your fix for the problem, implemented in this PR?

The placeholder is updated.

Make sure the following tasks are checked

@sambostock sambostock changed the title Replace "prior to" with "immediately after" Replace "prior to" with "immediately after" in bundle gem README.md.tt Feb 3, 2023
Copy link
Member

@deivid-rodriguez deivid-rodriguez left a comment

Choose a reason for hiding this comment

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

Thank you!

@simi
Copy link
Member

simi commented Feb 4, 2023

Would you mind to add this to this list next to the old name as well @sambostock?

sambostock added a commit to sambostock/rubygems.org that referenced this pull request Feb 4, 2023
@sambostock
Copy link
Contributor Author

@simi done in rubygems/rubygems.org#3392

@sambostock sambostock force-pushed the tweak-bundle-gem-readme-placeholder branch from 560fa87 to dabece5 Compare February 4, 2023 23:13
Currently, the instructions and placeholder contradict each other.

The commit that introduced this placeholder (42bc471)
makes it clear that the placeholder should be replaced AFTER publishing
(i.e. once the author successfully claims the gem name on Rubygems), so
the placeholder should match.
@sambostock sambostock force-pushed the tweak-bundle-gem-readme-placeholder branch from dabece5 to 23e2dea Compare February 4, 2023 23:14
@simi
Copy link
Member

simi commented Feb 5, 2023

I'll merge once the change is deployed to rubygems.org.

ℹ️ UPDATE: deployed

@simi simi enabled auto-merge February 5, 2023 01:08
@simi simi merged commit fbbfe89 into rubygems:master Feb 5, 2023
@sambostock sambostock deleted the tweak-bundle-gem-readme-placeholder branch February 6, 2023 01:53
@deivid-rodriguez deivid-rodriguez changed the title Replace "prior to" with "immediately after" in bundle gem README.md.tt Replace "prior to" with "immediately after" in bundle gemgenerated README file Dec 7, 2023
@deivid-rodriguez deivid-rodriguez changed the title Replace "prior to" with "immediately after" in bundle gemgenerated README file Replace "prior to" with "immediately after" in bundle gem generated README file Dec 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants