Repeated indentation

[phenomnomnominal]

Is this test actually testing the expected behaviour:

it("works with removing same number of spaces", () => {
  const dd = require("../dedent");
  const result = dd`
                    first
                       second
                          third
                   `;
  expect(result).toBe("first\nsecond\n   third");
});

I thought this was a bug, and have a fix for it, but then there's a test for it.

I would have expected the result to be "first\n___second\n______third" (MD is eating up my whitespace).

Let me know if it is actually a bug and I'll make a PR

[dmnd]

In retrospect that test does look pretty surprising. @pratapvardhan, thoughts?

[phenomnomnominal]

Any time you have a blank line, then a single indented line, then any number of subsequent lines with more indentation, the original indentation is wrong.

eg:

    some text
      more text
      more text

will become

some text
more text
more text

[phenomnomnominal]

Here's a PR: #9

[phenomnomnominal]

Progress on this? I'd rather depend on a real version that my fork.

[phenomnomnominal]

Bump?

[dmnd]

Sorry, I'll merge & release this next chance I get. Probably 2 days from now

[phenomnomnominal]

@dmnd sorry to bug you again, any chances of this happening soon?

[dmnd]

Sorry for the (huge) delay. Merged your PR! Thank you.