Pete Gonzalez

Results 561 comments of Pete Gonzalez

> TypeScript is able to differentiate between values and types of the same name. In this case `const MWSApiError` and `type MWSApiError` are two different things and are neither self-referential...

> IMO we will see more and more of these libraries and more use of them in the long run. So the tooling should be able to adapt and not...

@moltar If you get a chance to investigate this and provide more technical details, please share what you find.

We discussed this and I think arrived at two separate decisions: 1. **Create a library that performs accurate transforms.** `PnpmfileConfiguration.ts` should be split into two separate APIs: One half is...

@iclanton @dmichon-msft Are we ready to merge this?

> The expected outcome in this situation is that running api-extractor in `package-b` should trigger the `ae-incompatible-release-tags` error, but it doesn't. I'm not sure about that. `bundledPackages` is one of...

We encountered this problem with https://github.com/microsoft/rushstack/issues/1856 API Extractor generates report files that should never be processed by Prettier. It is possible to exclude them by file extension using `.prettierignore`. But...

> [`prettier-ignore-start` / `-end`](https://prettier.io/docs/en/ignore.html#range-ignore) should work in your case in Markdown. That's exactly what it was designed for. Interesting, yeah it does seem to work! And if I put ``...

+1 For certain problems, individual characters and their counts can be important, so ligatures directly interfere with my work. Whereas they can be fun and "zen-like" for people whose work...

The feature that we designed is part of [@rushstack/eslint-patch](https://www.npmjs.com/package/@rushstack/eslint-patch#eslint-bulk-suppressions-feature). Kevin Yang summarized it in [this blog post](https://developers.tiktok.com/blog/bulk-suppressions-a-new-eslint-feature-for-large-codebases). A key design question is how to keep bulk suppressions stable across Git...