Skip to content

Conversation

@liarco
Copy link

@liarco liarco commented Sep 24, 2021

This PR fixes #37.

Implementation details:

  • the comment parser now returns a third array value with the number of new lines at the end of the given comment
  • the initialization function runs the parser before setting the new lines count and EOL format in order to base its decisions on the template file content (if available)
  • tests have been updated to support the new parser feature and check that both UNIX and Windows EOLs work properly

I hope that this can be useful, please feel free to ask for changes or any further information.

Thank you for your time.

@liarco
Copy link
Author

liarco commented Sep 30, 2021

Hi @Stuk, I'm tagging you just to check if this project is still active or not.

Thank you for your time.

@liarco
Copy link
Author

liarco commented Oct 18, 2021

Hi @Stuk, I'm sorry to bother you, but do you have any news about this?

@max-kuzmin
Copy link

max-kuzmin commented Nov 2, 2021

@Stuk This is really useful feature

@liarco
Copy link
Author

liarco commented Nov 2, 2021

Thank you @MaxGsomGsom, unfortunately I'm not receiving any feedback from @Stuk. Maybe he's busy or doesn't have the notifications enabled.

@annkupriyanova
Copy link

annkupriyanova commented Nov 3, 2021

Cool feature! Thanks, @liarco

@tonyganchev
Copy link

tonyganchev commented Oct 24, 2025

I don't think @Stuk is active with this project in the last few years.

@liarco I am willing to pull this change in [my fork] but since this is a breaking change for existing repos, I'd plan it for v4. Would you file a corresponding issue there so that we can discuss the design to align with other things I want to add?

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.

Unable to specify a new line after a header using a template file

4 participants