Skip to content

Conversation

dacky179
Copy link
Member

@dacky179 dacky179 commented Aug 1, 2025

Port #550

1nf0rmagician and others added 16 commits November 21, 2024 14:00
…d-validation

Allow DeadEndValidator to pass on workplans with disconnected connectors
Test projects now reference `coverlet.collector` to create test coverage
reports.

Version 5.8.0 is used since newer versions result in build errors because some
dependency to `Newtonsoft.Json` couldn't be resolved.

Notice, that 5.8 does not exist and gets resolved to 6.0.0 instead. But using
6.0.0 gave the same dependency issues described above.

Providing an output path doesn't seem to work as before, so test files get
copied manually now.

The artifacts plugin was configured to report an error if not test results were
found: No test results found does either mean there are not tests in the
project, so no need to run this step anyway or that for some reason those
results could not be generated which is considered an error
…test-steps

Fix test steps in build pipeline
that will be the same after certificate renew
Make sure to always stop non periodical execution.
Make sure to always stop non periodical execution.
@CLAassistant
Copy link

CLAassistant commented Aug 1, 2025

CLA assistant check
All committers have signed the CLA.

@Toxantron
Copy link
Member

Proposal since Moryx Framework 3 is out of support anyway: Instead of merging the branches, could you create a new branch based on release/6 and cherry-pick just the ParallelOperations changes. @chaostrichter changes for the old pipeline and maintenance would break the build scripts.

@1nf0rmagician 1nf0rmagician self-requested a review September 9, 2025 04:03
@1nf0rmagician 1nf0rmagician added bug Something isn't working port Indicates that the changes in this PR were originally discussed and merged in a prior release labels Sep 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working port Indicates that the changes in this PR were originally discussed and merged in a prior release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants