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

Syntax of Line in New-MDQuote #33

Closed
belibug opened this issue May 30, 2024 · 3 comments · Fixed by #34
Closed

Syntax of Line in New-MDQuote #33

belibug opened this issue May 30, 2024 · 3 comments · Fixed by #34
Labels

Comments

@belibug
Copy link
Contributor

belibug commented May 30, 2024

This is how it currently works

 New-MDQuote -Lines @("Line 1","Line 2")

> Line 1
> 
> Line 2

Why the blank line between Line1 and Line2.

I checked the syntax guide here which doesn't have multiline example.

This is another doc link with multi line and nested quotes. They don't required empty "> " line between lines either.

@Sarafian
Copy link
Owner

I'm not sure. One hypothesis I can make is that when this was developed, one markdown rendered didn't play nice and I had to do it like that. Could be just as well me being silly and this is bug. Let's go with the bug.

@Sarafian
Copy link
Owner

The issue seems to be with the following lines

            if($output -ne "")
            {
                $output+="$prefix"+[System.Environment]::NewLine
            }

I'm not sure what was the reasoning behind this.

@Sarafian
Copy link
Owner

Let's go with the bug.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants