Skip to content

Improvements to CI pipeline#4248

Merged
pwojcikdev merged 15 commits intonanocurrency:developfrom
pwojcikdev:ci-overhaul-rebased
Jun 21, 2023
Merged

Improvements to CI pipeline#4248
pwojcikdev merged 15 commits intonanocurrency:developfrom
pwojcikdev:ci-overhaul-rebased

Conversation

@pwojcikdev
Copy link
Copy Markdown
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 requested a review from clemahieu June 21, 2023 10:04
@pwojcikdev pwojcikdev merged commit d33803e into nanocurrency:develop Jun 21, 2023
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.

2 participants