Skip to content
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

Improvements to CI pipeline #4248

Merged
merged 15 commits into from Jun 21, 2023

Conversation

pwojcikdev
Copy link
Contributor

@pwojcikdev pwojcikdev commented Jun 20, 2023

This PR is a collection of several improvements I made to the scripts in our CI pipeline. The focus was on simplification and avoiding duplicated logic between platforms.

  • Separated running Core / RPC / System Tests in GH workflow, so it's now clear which testsuite failed without having to scroll through log files
  • Made dependency install scripts simpler and more uniform.
  • Simplified docker image building with a proper multi-stage dockerfile. It is no longer needed to build a separate image(s) containing build environment(s).
  • Windows uses the same set of bash scripts as Linux/macOS. Additionally, System Tests are now running on Windows too.
  • Fixed parallel building flags. Parallel building is now able to use however many cores the underlying hardware provides.

@pwojcikdev pwojcikdev merged commit d33803e into nanocurrency:develop Jun 21, 2023
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants