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

Do not prevent the built-in FSMonitor patch series from being sent #619

Merged
merged 2 commits into from
Jun 30, 2021

Conversation

dscho
Copy link
Member

@dscho dscho commented Jun 30, 2021

Currently, gitgitgadget/git#923 is blocked because it has more commits than the (hard-coded) limit. Let's lift this limit for this PR specifically, and set an example how the same can be done in the future for other PRs.

Some patch series just _are_ long.

We could potentially add ways to exempt PRs via the GitGitGadget options
stored in the Git notes under `refs/notes/gitgitgadget`, but a much more
public way is to simply open a PR in the future that adds the exemptions
directly to the GitGitGadget code base ;-)

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
This patch series simply _is_ that long...

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Copy link

@jeffhostetler jeffhostetler left a comment

Choose a reason for hiding this comment

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

Nice! Thanks!

@derrickstolee derrickstolee merged commit 41b25d8 into gitgitgadget:main Jun 30, 2021
@dscho dscho deleted the exception-for-long-pr-923 branch June 30, 2021 19:16
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.

None yet

3 participants