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

Resolve build errors on linux #49

Merged
merged 6 commits into from
Oct 27, 2022
Merged

Resolve build errors on linux #49

merged 6 commits into from
Oct 27, 2022

Conversation

h-man2
Copy link
Contributor

@h-man2 h-man2 commented Oct 26, 2022

When run on linux, the integration tests failed because of additonal whitespaces.

In this PR the following changes have been implemented:

Ignore whitespace-only diffs

If the expected and actual output of an integrationtest only differ in whitespace changes the test does not fail.

Improved output of diffa

The output was somehow enriched with more detail

Build on Linux

Pipeline is now run on linux.

- Set version to 7.1.0
- Used same property for lineendings
- Switched pipeline to Linux
- Run pipeline after every push
- Improved error reporting
- Ignore whitespace-only diffs
@h-man2 h-man2 merged commit 23462fe into master Oct 27, 2022
@h-man2 h-man2 deleted the build_error_linux branch October 27, 2022 13:20
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.

1 participant