Skip to content

Conversation

@Steffengreiner
Copy link
Contributor

What was changed
This PR allows project spaces with hyphens in their name by removing their automatic replacement with underscores in the project space class

Additional Information
See DM-481 for more information

@Steffengreiner Steffengreiner requested a review from a team as a code owner August 26, 2022 09:04
Copy link
Contributor

@KochTobi KochTobi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Steffengreiner Steffengreiner merged commit 4757d9c into development Aug 26, 2022
@Steffengreiner Steffengreiner deleted the patch/dm-481-allow-hyphenated-project-space branch August 26, 2022 09:06
sven1103 pushed a commit that referenced this pull request Aug 26, 2022
* increase snapshot to 2.20.0

* Adjust Nf-core pipeline result to account for sarekv3 file structure

* Remove replacement of hyphens with underscores in project spaces (#326)

Co-authored-by: wow-such-code <[email protected]>
Steffengreiner added a commit that referenced this pull request Oct 13, 2022
* increase snapshot to 2.20.0

* Adjust Nf-core pipeline result to account for sarekv3 file structure

* Remove replacement of hyphens with underscores in project spaces (#326)

* add digiwest property tpes

* Support nanopore data structure v3 (#328)

* add new version of valid nanopore structure

* implement additional v3 changes

* Update schema and test to current file structure

Co-authored-by: Steffengreiner <[email protected]>

* Fix Nanopore Schema V3 naming and streamline children of other report definition

* Adapt schema and add test case for mibi datastructure

Co-authored-by: wow-such-code <[email protected]>
Steffengreiner added a commit that referenced this pull request Oct 13, 2022
* Prepare Version 2.20.0 (#325)

* increase snapshot to 2.20.0

* Adjust Nf-core pipeline result to account for sarekv3 file structure

* Remove replacement of hyphens with underscores in project spaces (#326)

Co-authored-by: wow-such-code <[email protected]>

* Update version to 2.20.0 (#327)

* Set version to 2.20.0

* remove unnessary whitespace

Co-authored-by: Steffengreiner <[email protected]>

* Release/2.21.0 (#331)

* increase snapshot to 2.20.0

* Adjust Nf-core pipeline result to account for sarekv3 file structure

* Remove replacement of hyphens with underscores in project spaces (#326)

* add digiwest property tpes

* Support nanopore data structure v3 (#328)

* add new version of valid nanopore structure

* implement additional v3 changes

* Update schema and test to current file structure

Co-authored-by: Steffengreiner <[email protected]>

* Fix Nanopore Schema V3 naming and streamline children of other report definition

* Adapt schema and add test case for mibi datastructure

Co-authored-by: wow-such-code <[email protected]>

* Set version to 2.21.0 (#332)

Co-authored-by: wow-such-code <[email protected]>
Co-authored-by: Johnny Q5 <[email protected]>
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.

3 participants