Skip to content

Commit d376184

Browse files
authored
chore(ci): add ci-success step so we can make this a condition for merging (#111)
<!-- Thank you for your Pull Request. Please provide a description above and review the requirements below. Bug fixes and new features should include tests. Contributors guide: https://github.com/foundry-rs/foundry-fork-db/blob/main/CONTRIBUTING.md The contributors guide includes instructions for running rustfmt and building the documentation. --> <!-- ** Please select "Allow edits from maintainers" in the PR Options ** --> ## Motivation <!-- Explain the context and why you're making that change. What is the problem you're trying to solve? In some cases there is not a problem and this can be thought of as being the motivation for your change. --> Adds `ci-success` similar to how we have in Foundry already to allow selecting it as a condition for merging ## Solution <!-- Summarize the solution and provide any necessary context needed to understand the code change. --> A PR will follow to further harden the workflow
1 parent ff83e44 commit d376184

File tree

1 file changed

+18
-0
lines changed

1 file changed

+18
-0
lines changed

.github/workflows/ci.yml

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -105,3 +105,21 @@ jobs:
105105

106106
deny:
107107
uses: ithacaxyz/ci/.github/workflows/deny.yml@main
108+
109+
ci-success:
110+
runs-on: ubuntu-latest
111+
if: always()
112+
needs:
113+
- test
114+
- wasm
115+
- feature-checks
116+
- clippy
117+
- docs
118+
- fmt
119+
- deny
120+
timeout-minutes: 30
121+
steps:
122+
- name: Decide whether the needed jobs succeeded or failed
123+
uses: re-actors/alls-green@release/v1
124+
with:
125+
jobs: ${{ toJSON(needs) }}

0 commit comments

Comments
 (0)