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

Update badge URL for Travis #18

Merged
merged 1 commit into from
Oct 28, 2020
Merged

Conversation

jugmac00
Copy link
Member

@mgedmin
Copy link
Member

mgedmin commented Oct 28, 2020

These are trivial and have zero chance of breaking CI. Does the tooling allow pushing directly to master rather than opening 100+ PRs?

@dataflake dataflake merged commit ef9b537 into master Oct 28, 2020
@dataflake dataflake deleted the all-repos_autofix_all-repos-sed branch October 28, 2020 16:17
@jugmac00
Copy link
Member Author

These are trivial and have zero chance of breaking CI. Does the tooling allow pushing directly to master rather than opening 100+ PRs?

Yes.

The tool is quite modular, and you even can create your own modules, but merging and pushing to master is builtin.

https://github.com/asottile/all-repos#all_repospushmerge_to_master

That said, I am not sure whether pushing to master is possible in all of Zopefoundation's repositories.

Certainly it is a good idea to discuss how to do a possible next mass pr creation.

@mgedmin
Copy link
Member

mgedmin commented Oct 29, 2020

That said, I am not sure whether pushing to master is possible in all of Zopefoundation's repositories.

It is: branch protections break zest.releaser's workflow.

We mostly don't push directly to master because we want to verify the changes (see CI passing, get reviewer feedback) before integrating them into master.

@icemac
Copy link
Member

icemac commented Oct 30, 2020

See also the comment of @dataflake to zopefoundation/meta#1.

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.

4 participants