Skip to content

Add script for cleaning#1003

Closed
StopMotionCuber wants to merge 2 commits intopulp:mainfrom
StopMotionCuber:clean-gh-release-notes
Closed

Add script for cleaning#1003
StopMotionCuber wants to merge 2 commits intopulp:mainfrom
StopMotionCuber:clean-gh-release-notes

Conversation

@StopMotionCuber
Copy link
Contributor

This is a follow up of pulp/pulpcore#7279, with the suggestions from @pedro-psb applied here

@StopMotionCuber
Copy link
Contributor Author

Also I gotta mention the Python script is not pretty per se, but gets the job done

Copy link
Member

@pedro-psb pedro-psb left a comment

Choose a reason for hiding this comment

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

There are some linting errors.
I'm not sure if I'll have to allow the workflow again... I'll be watching

version_str = match.group(1)
print("")
print("> [!NOTE]")
print(f"> Changes are also available on [Pulp docs](https://pulpproject.org/pulpcore/changes/#{version_str})")
Copy link
Member

Choose a reason for hiding this comment

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

This runs against all Pulp plugins, so you can't hardcode pulpcore here.
You can either make this script a template (.j2) and use {{ plugin_name }} or pass as an argument trough publish.yml.j2, which is already a template.

@pedro-psb
Copy link
Member

@StopMotionCuber I took the liberty to create another PR based on this one and add you as co-author: #1009
Maybe I close this one?

@StopMotionCuber
Copy link
Contributor Author

Oh sure. I'm sorry I didn't follow up on this one, but started this work when I was at cfgmgmtcamp, as it was a low hanging fruit, but then forgot about it afterwards when there was backlog from work.

Thanks for following up on this one

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.

2 participants