Pete Gonzalez

Results 684 comments of Pete Gonzalez
trafficstars

> For API members documented with `{@inheritDoc Foo}`, the resulting generated API report marks these as `// undocumented`. This isn't true, and leads to confusion in PRs, etc. It would...

> This is no longer the case, right? From what I recall the native dependencies came from the Identity package which has now been resolved Yes, we have been working...

Making it optional sounds reasonable. 👍 I'm curious what it gets in the way of, though? Another idea would be for reprinting to be triggered by some keystroke such as...

Can you share a repro branch? We're very close to releasing AE7, so this week I'm going through and retesting various projects to find any remaining blocking bugs. I wonder...

Agreed we should probably emit a better name than `default`. But your code is mixing together two different ways of importing React: ```ts import React from "react"; ``` ```ts import...

Notice that @elenasch cited **Rush 5.112.2** which is from December. Is this really a regression? Maybe it is just an edge case. I'm actually unclear what `rush install --only .`...

It might make sense to forbid the unsafe selectors (`--only`, `--impacted-by`, `--impacted-by-except`) for `rush install` and `rush update`. @dmichon-msft what do you think?

Probably not at this time. It seems that Bun implements the [flawed NPM installation model](https://lfx.rushstack.io/pages/concepts/install_models/). For example, I tried converting the [demo/sbs-3](https://lfx.rushstack.io/pages/scenarios/side-by-side_versions/#step-3-side-by-side-version-trouble) branch to use a Bun workspace. Its dependency...

Apparently Bun only supports Windows Subsystem for Linux (WSL) not actual Windows/NTFS, which might be a blocker for companies that ship device apps that build using Windows. This is less...