Sebastian Schuberth

Results 1424 comments of Sebastian Schuberth

> What's needed to get this merged? Two approvals are needed since recently. Maybe @pombredanne can also review?

Unfortunately, @Yannic you have read-only permissions, so the approval does not count towards mergability: ![image](https://github.com/user-attachments/assets/4f79d299-efd6-49b4-8899-fe277a3447a8) ![image](https://github.com/user-attachments/assets/21b600cb-16e9-4995-9676-70526304022a)

> @sschuberth Do you happen to have write permission and could add the second approval yourself? @fmeum As mentioned over [here](https://github.com/package-url/packageurl-go/pull/76#issuecomment-2507737408) I'm currently pausing reviews to this project until some...

> @sschuberth do you have a rough estimate on when you expect the process questions to be resolved? A few days? A few weeks? I simply have no idea. I've...

More specifically, it would be great if `MapPropertySource` and `CommandLinePropertySource` would support the "key" to contain an index like `[1]` to be able to override single values nested within an...

@sksamuel I'm willing to work on this, but I have no clue where to start. Is my approach outlined above even feasible to implement?

While I didn't have time to work on this yet, I believe it still makes sense as a feature to add.

> aws preprocessor with [] used for json fields. Maybe by now https://github.com/eygraber/JsonPathKt could be leveraged...

> Not sure we even need that. Just need to take [n] and apply that to the list ? For lists you're probably right. But we basically have the same...

> I'm not sure what you were thinking about for maps. I didn't have a concrete syntax in mind yet. I only know our requirement to be able to set...