Anton Tayanovskyy

Notes to self. justin/patch branch for prior experiments. Used github.com/mattbaird/jsonpatch (114*) Also https://github.com/evanphx/json-patch/issues/56 From comments, there is https://github.com/herkyl/patchwerk (5*) https://github.com/evanphx/json-patch (785*) All these give options to support RFC6902 patches but...

Finding bugs with https://github.com/mattbaird/jsonpatch though maybe that's just how the code uses it. Hmm: ``` json_patch_test.go:121: [rapid] panic after 0 tests: error in remove for path: '/a/1': Unable to access...

Can't seem to repro on ``` CLI Version 3.39.2 Go Version go1.19 Go Compiler gc Plugins NAME VERSION aws 5.13.0 go unknown Host OS darwin Version 12.5.1 Arch x86_64 This...

Thanks for reporting this @rshade - this sounds important. For those of us who are a bit less familiar with this part of the product, could you provide some repro...

I've tried to reproduce the issue with a program in Go like this: ```go package main import ( "github.com/pulumi/pulumi-aws/sdk/v5/go/aws" "github.com/pulumi/pulumi-aws/sdk/v5/go/aws/s3" "github.com/pulumi/pulumi/sdk/v3/go/pulumi" ) func main() { pulumi.Run(func(ctx *pulumi.Context) error { //...

I've experimented some more here. It sounds full `pulumi up` and `pulumi refresh` work as expected. However if I understand correctly there is something in the user situation that a...

Related: https://github.com/pulumi/pulumi/issues/8613 Note that working on this issue may also help break down large output.d.ts files by moving definitions out somewhere else and re-importing them, what 8613 implies. However, it...

@RobbieMcKinstry I've published an utility that might help here https://github.com/t0yv0/tsc-resolution-tracker

👋 I'm late to review this, don't wait on me but if it's still open I'll review on Monday. Apologies.

Changes look allright, lots of tests, I was just wondering if: - this is intended to be breaking in some sense or strictly bw-compat (see comments above) - this is...