Skip to content
This repository was archived by the owner on Apr 11, 2021. It is now read-only.

fix/ci automation#100

Open
tynes wants to merge 1 commit intomasterfrom
fix/ci-automation
Open

fix/ci automation#100
tynes wants to merge 1 commit intomasterfrom
fix/ci-automation

Conversation

@tynes
Copy link
Copy Markdown
Collaborator

@tynes tynes commented Mar 12, 2021

Adds a fix for CI automation where it automatically opens PRs in the submodules. The dev workflow is to instead open PRs against the integration repo when developing with multiple submodules

@tynes
Copy link
Copy Markdown
Collaborator Author

tynes commented Mar 13, 2021

After working on a POC work an automated workflow using optimism-integration, I've determined the only way to make this nice is using a server that listens to Github webhooks and opens/merges PRs automatically. The developer workflow that we were going for is as follows:

  • any changes that require changes across multiple repos should be made against this repository by checking the submodules out to specific branches and committing those submodules
  • the CI handles automatically opening PRs to the submodules
  • when the PR in this repo is merged, the submodule PRs are merged automatically

For now, the solution is to make pull requests against this repository with the submodules checked out to the specific branches. The CI will run the checked out branches, we observe that the CI passes and then we manually make PRs and merges in the other repos

ci: automated pr opening

ci: remove submodule push

ci: auto pr

ci: auto pr fix

ci: sh

ci: fix for sh support

ci: fix for sh support

ci: bugfixing

ci: bugfixing

ci: bugfixing

ci: bugfixing

ci: bugfixing

ci: bugfixing draft

ci: bugfixing draft

ci: bugfixing auth

ci: bugfixing auth

ci: bugfixing title

ci: bugfixing title

ci: bugfixing title

ci: bugfixing nice body

ci: bugfixing artifacts

ci: bugfixing artifacts

ci: bugfixing

ci: bugfixing
@tynes tynes force-pushed the fix/ci-automation branch from 3ffe7ff to d8b8a48 Compare March 13, 2021 02:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant