[jest-docblock] Do not extract /* these */ comments

[azz]

Do you want to request a feature or report a bug?

Bug

What is the expected behavior?

expect(
  require('jest-docblock').extract(`
    /* a */
    /** b */
  `)
).toEqual(`/** b */`)

What is the current behavior?

It returns the non-docblock /* a */.

Fixing this could be considered a breaking change, so perhaps adding an option {strict:true} is preferable?

[SimenB]

Feels like a bug to me, not necessarily breaking. @cpojer?

[azz]

Looks like there's a test to the contrary.

https://github.com/facebook/jest/blob/64cbf36eb1c34f8839e484ae0da355e5edcf4ec2/packages/jest-docblock/src/__tests__/index.test.js#L69-L81

@edvinerikson might remember why? https://github.com/facebook/jest/pull/2012/files#diff-0f378680ff446d6e4f035dd680ab24e4R14

[sygint]

If anyone approves of a solution, I can write this

[SimenB]

@cpojer thoughts here?

[sygint]

anything? this still seems to be affecting other libraries like Prettier

[edvinerikson]

This was the reason #2005

[SimenB]

Seeing as @providesModule is out (#6104), maybe we can fix this?

[github-actions]

This issue is stale because it has been open for 1 year with no activity. Remove stale label or comment or this will be closed in 30 days.

[github-actions]

This issue was closed because it has been stalled for 7 days with no activity. Please open a new issue if the issue is still relevant, linking to this one.

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.