Skip to content

Document exportpatches [squash when merging] - #21

Open
befeleme wants to merge 4 commits into
fedora-python:masterfrom
befeleme:exporttoo
Open

Document exportpatches [squash when merging]#21
befeleme wants to merge 4 commits into
fedora-python:masterfrom
befeleme:exporttoo

Conversation

@befeleme

@befeleme befeleme commented Sep 3, 2026

Copy link
Copy Markdown
  • How to install (in parallel to importpatches)
  • That the script makes assumption about your remote's name

- How to install (in parallel to importpatches)
- That the script makes assumption about your remote's name

@hroncok hroncok left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I was confused that the remote name requirement is now listed in the importpatches.py section when it is needed for exportpatches.py. But there is no exportpatches.py section :D

Comment thread README.md Outdated
@befeleme

befeleme commented Sep 4, 2026

Copy link
Copy Markdown
Author

I made more logical changes to readme so it reflects the reality of both scripts existing.
Do we want also a section explaining what exportpatches does exactly?

@befeleme befeleme changed the title Document exportpatches Document exportpatches [squash when merging] Sep 4, 2026
@befeleme

befeleme commented Sep 4, 2026

Copy link
Copy Markdown
Author

Do we want also a section explaining what exportpatches does exactly?

Actually, robots did that in a minute, see latest commit.

@hrnciar

hrnciar commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

+1, thank you

@hroncok

hroncok commented Sep 4, 2026

Copy link
Copy Markdown
Member

robots did that in a minute

Let's disclose robots usage in the (squashed) commit message? Technically, I think https://docs.fedoraproject.org/en-US/council/policy/ai-contribution-policy/ applies

@hroncok hroncok left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The changes look good to me. Please squash with LLM Assisted-By trailer (assuming I understood the robots comment correctly).

@hroncok hroncok mentioned this pull request Sep 4, 2026
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