Skip to content

Fail fast when results-dir is incorrect#772

Merged
jan-goral merged 4 commits into
masterfrom
753-fail-fast-incorrect-results-dir
May 7, 2020
Merged

Fail fast when results-dir is incorrect#772
jan-goral merged 4 commits into
masterfrom
753-fail-fast-incorrect-results-dir

Conversation

@jan-goral
Copy link
Copy Markdown
Contributor

@jan-goral jan-goral commented May 7, 2020

Fixes #753

Checklist

  • Unit tested
  • release_notes.md updated

@jan-goral jan-goral self-assigned this May 7, 2020
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #772 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #772   +/-   ##
=========================================
  Coverage     78.22%   78.22%           
  Complexity      665      665           
=========================================
  Files           125      125           
  Lines          2778     2778           
  Branches        399      399           
=========================================
  Hits           2173     2173           
  Misses          351      351           
  Partials        254      254           

bootstraponline
bootstraponline previously approved these changes May 7, 2020
@jan-goral jan-goral merged commit 148c59b into master May 7, 2020
@jan-goral jan-goral deleted the 753-fail-fast-incorrect-results-dir branch May 7, 2020 20:36
adamfilipow92 pushed a commit that referenced this pull request May 8, 2020
* Fail fast when results-dir is incorrect
adamfilipow92 added a commit that referenced this pull request May 8, 2020
* #764

Fix crash on parse some control chars

* #764 detekt issues fix

detekt issues fix

* Asserts in tests

Add asserts in test and replace Files.readString to Files.readAllBytes for compability with java 1.8

* Add documentation of magicial numers

* set name of UtfControlChars enum to UtfControlCharRanges

* #764

Fix crash on parse some control chars

* #764 detekt issues fix

detekt issues fix

* Asserts in tests

Add asserts in test and replace Files.readString to Files.readAllBytes for compability with java 1.8

* Add documentation of magicial numers

* set name of UtfControlChars enum to UtfControlCharRanges

* Detekt suggestions

* Create should_exists.txt

* #764

Fix crash on parse some control chars

* #764 detekt issues fix

detekt issues fix

* Asserts in tests

Add asserts in test and replace Files.readString to Files.readAllBytes for compability with java 1.8

* Add documentation of magicial numers

* set name of UtfControlChars enum to UtfControlCharRanges

* Fail fast when results-dir is incorrect (#772)

* Fail fast when results-dir is incorrect

* #764 Change XmlPreprocessor more functional and

remove should_exists

* Add info about issue to release notes

Co-authored-by: Adam <adam.filipowicz92@gmail.com>
Co-authored-by: Jan Góral <60390247+jan-gogo@users.noreply.github.com>
adamfilipow92 added a commit that referenced this pull request May 13, 2020
* #764

Fix crash on parse some control chars

* #764 detekt issues fix

detekt issues fix

* Asserts in tests

Add asserts in test and replace Files.readString to Files.readAllBytes for compability with java 1.8

* Add documentation of magicial numers

* set name of UtfControlChars enum to UtfControlCharRanges

* #764

Fix crash on parse some control chars

* #764 detekt issues fix

detekt issues fix

* Asserts in tests

Add asserts in test and replace Files.readString to Files.readAllBytes for compability with java 1.8

* Add documentation of magicial numers

* set name of UtfControlChars enum to UtfControlCharRanges

* Detekt suggestions

* Create should_exists.txt

* #764

Fix crash on parse some control chars

* #764 detekt issues fix

detekt issues fix

* Asserts in tests

Add asserts in test and replace Files.readString to Files.readAllBytes for compability with java 1.8

* Add documentation of magicial numers

* set name of UtfControlChars enum to UtfControlCharRanges

* Fail fast when results-dir is incorrect (#772)

* Fail fast when results-dir is incorrect

* #764 Change XmlPreprocessor more functional and

remove should_exists

* Add info about issue to release notes

Co-authored-by: Adam <adam.filipowicz92@gmail.com>
Co-authored-by: Jan Góral <60390247+jan-gogo@users.noreply.github.com>
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.

Fail fast when results-dir string is incorrect

4 participants