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

Outline the bugfix backport process #1092

Merged
merged 1 commit into from
Dec 1, 2023
Merged

Conversation

mdellweg
Copy link
Member

[noissue]

README.rst Outdated

Since this plugin has been moved to pulpcore, issues can no longer be reported here. Please go to
`Pulpcore - Issues<https://github.com/pulp/pulpcore/issues>`_ instead.
Any issues that asrise should also be solved there first and if needed backported to here. We do
Copy link
Member

Choose a reason for hiding this comment

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

s/asrise/arise

Copy link
Member

Choose a reason for hiding this comment

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

s/backported to here/backported here

Copy link
Member

Choose a reason for hiding this comment

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

I would drop last 3 sentences

README.rst Outdated
Comment on lines 20 to 22
Any issues that arise should also be solved there first and if needed backported to here. We do
neither expect this to happen soon, nor often. So we will work out the details when / if that
happens. Please excuse if this takes more time than estimated.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Any issues that arise should also be solved there first and if needed backported to here. We do
neither expect this to happen soon, nor often. So we will work out the details when / if that
happens. Please excuse if this takes more time than estimated.
Any issues that arise should also be solved there first and if needed backported to here. We expect this to happen only infrequently, if ever. We will therefore work out the details when / if it
happens. Please excuse us if this takes more time than desired.

Copy link
Member Author

Choose a reason for hiding this comment

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

@ipanova suggested to drop that part. WDYT?

Copy link
Member

Choose a reason for hiding this comment

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

I suggested to drop it because user is not really interested how we solve this and what are our expectations. If there is a case for the backport in this repo we will handle it. That's all he's interested in, imo.

Copy link
Contributor

Choose a reason for hiding this comment

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

Dropping it sounds good, for exactly @ipanova 's reasoning

@ggainey ggainey merged commit b2fedec into pulp:main Dec 1, 2023
10 checks passed
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