-
Notifications
You must be signed in to change notification settings - Fork 21
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
Reuse jobs of existing Github Actions Workflow #20
base: main
Are you sure you want to change the base?
Conversation
@PierreZ can we please run this to see if it's working the right way? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @mohitsaxenaknoldus, this looks really nice, nice work 🥇
I left a couple of comments, mostly in order to split the work, and make the steps useable for other developers and workflows.
Feel free to push some commits 🚀
Unfortunately, I could run it, but it won't work, as we are still impacted by the official FDB site downtime. Downloading FDB from the official website is like the first step for every workflows 😞 I'm checking everyday the relevant issue and also the related posts on the forum, but no ETA for now. |
Done. |
Is it working now? |
Website is up since yesterday, but artifacts have been moved. We are working on publishing a new version for the Github Action. Once this is published, we will merge #19, and everything should work. I'll run the action once everything is ready. In the mean time, I think you have some indentation errors on your yaml, as Github is not setting some keys in green |
Should be fixed now, thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CI is now fixed, please rebase your PR. I also left some comments.
Pull Request Test Coverage Report for Build 1802989275
💛 - Coveralls |
Codecov Report
@@ Coverage Diff @@
## main #20 +/- ##
==========================================
+ Coverage 81.49% 83.43% +1.93%
==========================================
Files 18 24 +6
Lines 3091 4714 +1623
==========================================
+ Hits 2519 3933 +1414
- Misses 572 781 +209
Continue to review full report at Codecov.
|
I think something is not working somewhere: https://github.com/foundationdb-rs/foundationdb-rs/actions/runs/1717931608
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
error parsing called workflow "foundationdb-rs/foundationdb-rs/.github/workflows/correctness.yml@main": failed to fetch workflow: workflow was not found.
This is supposed to be the right syntax :/ |
This is true, the file does not yet exists on the main branch, only on your branch. Maybe you could resolve it locally(i.e in your own branch?) |
Fixes #18