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

Fix diff content for results in delta reports #650

Merged
merged 4 commits into from Jul 26, 2019

Conversation

@bjoernricks
Copy link
Member

commented Jul 25, 2019

Checklist:

@bjoernricks bjoernricks marked this pull request as ready for review Jul 25, 2019
bjoernricks added 4 commits Jul 25, 2019
Just free the data directly after it has been used.
For result diffs in delta reports spaces and newlines don't have a
meaning and should be ignored.
@bjoernricks bjoernricks force-pushed the bjoernricks:fix-diff-content branch from 78b8da1 to 23e6702 Jul 25, 2019
@mattmundell mattmundell merged commit 4776c11 into greenbone:master Jul 26, 2019
12 checks passed
12 checks passed
build Workflow: build
Details
ci/circleci: build_doc Your tests passed on CircleCI!
Details
ci/circleci: build_postgresql_debug Your tests passed on CircleCI!
Details
ci/circleci: build_postgresql_debug_clang Your tests passed on CircleCI!
Details
ci/circleci: build_postgresql_release Your tests passed on CircleCI!
Details
ci/circleci: check_format Your tests passed on CircleCI!
Details
ci/circleci: doc_coverage Your tests passed on CircleCI!
Details
ci/circleci: gen_xml_doc Your tests passed on CircleCI!
Details
ci/circleci: scan_build_postgresql_debug Your tests passed on CircleCI!
Details
ci/circleci: test_units Your tests passed on CircleCI!
Details
codecov/patch Coverage not affected when comparing c19cfce...23e6702
Details
codecov/project 92.1% remains the same compared to c19cfce
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.