ci: fix Diamond Storage Check artifact upload#1007
ci: fix Diamond Storage Check artifact upload#1007energypantry wants to merge 2 commits intoubiquity:developmentfrom
Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
All checks are green now. @rndquu when you have a moment, could you please review/merge this fix for the Diamond Storage Check artifact-name failure? Thanks. |
|
Quick follow-up: all checks are green on this PR. When you have a moment, could you please review/merge the CI fix so Diamond Storage Check stops failing on PRs? Thanks. cc @rndquu |
|
@rndquu Could you take a quick Codeowner look on this PR when you have a moment? All checks are green and it's ready to merge once the requested review is in. Happy to tweak anything you spot. |
Fixes #992.
The Diamond Storage Check workflow was failing with an invalid artifact name error (artifact name generated from branch + contract path).
This PR switches the workflow to a patched foundry-storage-check action revision that:
Action ref used:
energypantry/foundry-storage-check@fb95d25056453a7d61643f411ab2b3acaea33a39.If you prefer keeping the action under the
ubiquityorg, I can open the corresponding upstream PR (same commits) toubiquity/foundry-storage-checkand then we can repin to that ref.