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

Adhere to Prettier formatting #2021

Merged
merged 2 commits into from
Nov 15, 2022
Merged

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.

3 participants