-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Merge stable into main; Release 8.2.0
#2873
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
Conversation
for more information, see https://pre-commit.ci
… into trim_args_section
…m_args_section
Co-authored-by: Andreas Backx <andreas@backx.org>
Co-authored-by: Andreas Backx <andreas@backx.org>
Remove the per-project files so we don't have to keep them in sync. GitHub's UI links to everything except the contributing guide, so add a section about that to the readme.
stable into mainstable into main; Release 8.2.0
|
All tests are passing, branches have been merged, this PR is ready for a 8.2.0 release of Click! 🚀 |
Rowlando13
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I reviewed all the docs and requirements file changes.
|
@kdeldycke Can you make a PR for this branch against stable to show that it resolves all the conflicts from stable into main, unless there is an easier way to show that. |
|
Also thanks for this great PR! I wanted to fix the GitHub actions issues last week but did not get around to it. |
I already had a hard time reconciling the two branches so not sure I can find a clean way. But I'll try. |
I'm not sure I understand your request, as this PR is already merging with |
|
This shows what I wanted. Thanks. |
|
@kdeldycke Can you please fix the conflicts. @davidism Can you give this a review? |
Just fixed the merge conflicts. Is it possible to wait too long in merging the @AndreasBackx tried to fix it 6 months ago in #2830 but abandoned. I tried last month but I find it difficult to keep this PR open. Can you please help me reduce the entropy caused by the branch divergence? 😣 |
|
Yes. Absolutely. I was not planning on merging anything until this is merged. I thought that the merge conflict was missed in the original work. Sorry if I created by merging something. I was waiting for a code review since it is so complex, but in light of it possibly not getting done for a long time, I am going to merge it. |
|
I just checked the result of that merge and haven't found any issue. |
|
Sweet! |


In order to test the upcoming 8.2.0 release, I'm working on this PR that is trying to reconcile
stablebranch with themainbranch.Both branches have diverged a lot, and cannot be merged naively. This PR is fixing the conflicting changes which happened in both branches.
This PR particularly:
typingjobpre-commitworkflowhelp_option_names#2811--helpoption defaults from its class to its decorator #2840Relation to other proposals: