Pete Gonzalez
Pete Gonzalez
Here's my initial feedback: 1. How would this interact with the multiphase model that you proposed in #2300? Would we cache each phase as a separate tarball? Or would only...
Symlinks cannot be stored easily in a tarball. Rush should probably report an error if a symlink is encountered while archiving a file.
> with `@microsoft/rush-lib` `5.42.0` pnpm with `strictPeerDependencies` reports the following > ERROR: `@microsoft/rush-lib > @aws-sdk/client-s3 > @aws-sdk/middleware-retry: [email protected] requires a peer of react-native@>=0.56 but none was installed` @boardend This is...
> I'm going to create a separate PR to add the requested telemetry data to the default telemetry files. I believe it would be cleanest to then convert the analyzer...
> That would be useful. We'd have to handle the case when both a `.json` and `.jsonc` file exist. > > Would you be interested in putting this together? @iclanton...
YAML has some downsides: - The syntax is [crazy complicated](https://quickref.me/yaml) unless you restrict yourself to a subset. - It relies on JSON as its paradigm, but some YAML features can't...
@iclanton @D4N14L @apostolisms @elliot-nelson We seem to have gotten quite a lot of feedback about this topic in recent months (via a number of different tangents that start with *"I...
> I know that the reason we don't like `.js` for our config files is that at least parsing a JSON file can reasonably be expected to be bounded in...
> but still somehow downloads 80-some packages out of the 3k or so that we use each time we install on the CI (even when no changes to pnpm-lock.yaml are...
> This is a bit trickier to do with just the CI tool's caching mechanism and would probably need Rush to have some sort of cache implementation like Bazel or...