Skip to content

Commit 40b2448

Browse files
authored
chore(ci): add ci-success step so we can make this a condition for merging (#316)
<!-- 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 ## 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 9828708 commit 40b2448

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
@@ -109,3 +109,21 @@ jobs:
109109

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

0 commit comments

Comments
 (0)