Nathan Nguyen

Results 8 comments of Nathan Nguyen

How about using the `ctx.component.repository.commit_sha` field, which is not nullable, to replace the nullable `branch_name`? ```diff predicate["invocation"]["configSource"]["uri"] = ( f"{ctx.component.repository.remote_path}" - f"@refs/heads/{ctx.component.repository.branch_name}" + f"@{ctx.component.repository.commit_sha}" ) ``` I think this should...

After our discussion, I think we can update the list as follows: - [ ] Add help/description for each sub-command. - [ ] Make sure not to use the same...

I wonder if we could provide a [regex option](https://cuelang.org/docs/tutorials/tour/expressions/regexp/) to let users decide if they want to keep the target field version-agnostic. It may look like this, where `=~` is...

Can we utilize syrupy for this purpose? It already supports comparing complex JSON structures and overwriting existing results.

If we have non-deterministic parts in the expected result files, then can't we just "eliminate those non-deterministic parts" from these expected result files? These files can store just the "deterministic...

Yep. You're right that in that case, we need to cherry-pick things from the current output, and that part requires maintenance. Something I am wondering is whether the compare script...

I think this is related to #368.

On option 1: Removing `--config-path` completely. I would go for this option. If there is a need for it in the future (like you are alluding to in option 3),...