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

feature: Add configuration option for soft breaks #7

Closed
wants to merge 5 commits into from

Conversation

espenhogbakk
Copy link

This allows you to configure soft breaks to be rendered as "hard breaks" by passing in the whitespace you want to use in the options.

This allows you to configure soft breaks to be rendered as "hard breaks" by passing in the whitespace you want to use in the options.
@maiers
Copy link
Owner

maiers commented Dec 13, 2020

Hi @espenhogbakk,

thanks for the pull request. The configuration option for soft breaks looks fine to me.

But please do not commit the dist folder to git.

Can you update the pull request?

Cheers,
Sebastian

@maiers maiers assigned maiers and espenhogbakk and unassigned maiers Dec 13, 2020
@espenhogbakk
Copy link
Author

But please do not commit the dist folder to git.

Can you update the pull request?

I @maiers I will do, I totally forgot about that and I agree this should not be in this PR, but I would urge you to reconsider not having the dist folder in the repo. I do understand why you don't want to, but it does prevent one from installing this library directly from GitHub without relying on released packages on npm. Which is why I changed it so I could install the code in my PR before this was merged in and released.

@maiers
Copy link
Owner

maiers commented Jan 2, 2023

Closing because of inactivity.

@maiers maiers closed this Jan 2, 2023
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.

None yet

2 participants