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

Add manual format to typescript module factory #2471

Merged

Conversation

DamnClin
Copy link
Collaborator

No description provided.

@codecov
Copy link

codecov bot commented Jul 10, 2022

Codecov Report

Merging #2471 (d4eb6a7) into main (d5d7a33) will not change coverage.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##                main     #2471   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
  Complexity      2428      2428           
===========================================
  Files            549       549           
  Lines           9559      9559           
  Branches         231       231           
===========================================
  Hits            9559      9559           
Impacted Files Coverage Δ
...tor/typescript/domain/TypescriptModuleFactory.java 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d5d7a33...d4eb6a7. Read the comment docs.

@DamnClin
Copy link
Collaborator Author

@SoniaLahi I made a manual format here because we kinda agreed that it was more readable for this kind of module. Tell me if you disagree and I close this :)

@pascalgrimaud pascalgrimaud merged commit d947791 into jhipster:main Jul 11, 2022
@DamnClin DamnClin deleted the format-typescript-module-factory branch October 25, 2022 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants