Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@fabianegli
Copy link
Contributor

This PR makes the template adhere to the Prettier formatting style. Found during development on #1983

PR checklist

  • This comment contains a description of changes (with reason)

@edmundmiller
Copy link
Contributor

I forget if this breaks something, we should check the previous changes

@codecov
Copy link

codecov bot commented Nov 14, 2022

Codecov Report

Merging #2021 (0453dfd) into dev (564868f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #2021   +/-   ##
=======================================
  Coverage   63.86%   63.86%           
=======================================
  Files          41       41           
  Lines        4923     4923           
=======================================
  Hits         3144     3144           
  Misses       1779     1779           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@fabianegli
Copy link
Contributor Author

fabianegli commented Nov 14, 2022

This commit introduces both, the # { (line 24) and #{ (line 18) variant. I think it should be fine (based on running the prettier hook on the resulting module from nf_core modules create).

@fabianegli fabianegli changed the title adherer to prettier formatting Adhere to Prettier formatting Nov 14, 2022
@fabianegli fabianegli merged commit 6f0d96e into dev Nov 15, 2022
@fabianegli fabianegli deleted the prettier-formatting-in-module-template branch November 15, 2022 08:29
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.

4 participants