Karl Horky

Results 973 comments of Karl Horky

@emilyrohrbough any plans on the radar about fixing this problem out of the box for all Cypress TypeScript users?

Thanks for [the responses](https://github.com/npm/rfcs/blob/latest/meetings/2021-02-17.md#issue-325-rrfc-run-preinstall--postinstall-scripts-on-single-package-installation---karlhorky) during [the Open RFC Meeting on Feb 17, 2021](https://github.com/npm/rfcs/issues/326)! Interesting to hear a bit of the background. I added "Hook Scripts" to a new section above...

Thanks again for [the responses](https://github.com/npm/rfcs/blob/latest/meetings/2021-02-24.md) during [the Open RFC Meeting on Feb 24, 2021](https://github.com/npm/rfcs/issues/330)! Sorry for not being able to join again! The timing is kind of difficult for me...

Thanks also for [the responses](https://github.com/npm/rfcs/blob/latest/meetings/2021-03-03.md) during [the Open RFC Meeting on Feb 24, 2021](https://github.com/npm/rfcs/issues/335)! > @isaacs we need a "install tree has been mutated" lifecycle script, that would solve this...

Interesting, I think I may have found more information - during publishing it opens the browser to this URL, which is creating a new tag without the `v` prefix (2...

@mmkal maybe line 11 here? https://github.com/sindresorhus/np/blob/4b3b599571c9475b2be035b370fef4fe0b2e85d0/source/release-task-helper.js#L1-L18

> We probably should keep the no-prefix behaviour for pnpm Hm... my gut feel would be to add prefixes everywhere. Seems very surprising that if I use `pnpm`, I suddenly...

Yeah, good question why it's returning something different. 🤔 I would lean towards doing the change as a patch release (even if it's technically a breaking change) since the behavior...

Nice, I saw that #739 was reviewed and merged and released in [`[email protected]`](https://github.com/sindresorhus/np/releases/tag/v10.0.2), thanks @mmkal and @sindresorhus ! I just tested `[email protected]` with a project, and it's working as it...

## Workaround Wrapping the interpolation in parentheses works around the problem: ```js sql` SELECT * FROM longer_table_name_with_many_characters WHERE longer_table_name_with_many_characters.id IN ( ${sql(longerTableNameWithManyCharactersIds)} ) ` ``` But this workaround fails with...