Roman

Results 535 comments of Roman

Apparently there's a way to mark peer deps optional: https://docs.npmjs.com/cli/v11/configuring-npm/package-json#peerdependenciesmeta Although this may constitute a breaking change and a major bump as everyone who got peers automatically before will no...

I think it looks like pnpm is just not being nice! https://github.com/pnpm/pnpm/issues/5928

Throwing in my TypeScript solution for cross-region buckets based on the above: ```ts const { crossRegionSupport, artifactBucket } = pipeline.pipeline const artifactBuckets = [ artifactBucket, ...Object.values(crossRegionSupport).map((crs) => crs.replicationBucket), ] for...

Hm, still works for me!

Oh, I think I found prior art: ```yaml - uses: actions/upload-artifact@v4 with: name: repo.patch-${{ strategy.job-index }} ``` Not sure if the `strategy.job-index` is set and if it will fail if...

I find it extremely difficult to patch GitHub actions in a sane way. I think we need to expose some way to do this, or just add it to the...

> But committing the mutations from two build doesn't work either. WDYM? > What makes this different? It's not different. What I am doing with patching is also "extremely difficult"...

We would like to use JSON instead of YAML, simply because that is the default serialization format for JS-land. We use CDK for infra, which uses JSON, and providing a...

I was thinking the interface can be: ```ts interface ISecretProvider { bind: (project: Project, secretKey: string) => string } ``` Project could be useful for deriving some information from it,...

Used this filter to fix it: ```yaml OSPackage: - type: glob value: "G*" ``` ![screenshot-20231220T114041-JS8hdWFg@2x](https://github.com/rebuy-de/aws-nuke/assets/491247/8125760f-4154-4191-92c1-c688c8921b4b)