Victor

Results 18 comments of Victor

I see your point, but is it worth to add custom Marshal/Unmarshal functions? It will require to manually assemble all the other fields and can be error-prone. Using pointers when...

weekend again! what can I do to keep this moving?

What do you think about following the pattern of other packages to handle nullable fields? For example, in aws cli you can find the [String function](https://github.com/aws/aws-sdk-go/blob/master/aws/convert_types.go#L6) or the [Bool function](https://github.com/aws/aws-sdk-go/blob/master/aws/convert_types.go#L65)...

I understand that absent means the default value. I think that the key difference is that with the current implementation you can not tell the difference between the default value...

Hey, I'm probably not going to follow up with this MR, I'm currently focused on other projects. Maybe we should just close it.

I see, The point is that looks like the code is not up to date. Looking only at map defaults, it is defined right-down for RenderOrder, but other defaults are...

Maybe we can turn this issue into a task "Review all default values"

Hi @Qwarkster ! Welcome to the amazing Go community The visibility field is a clear good example. I also found that the editor doesn't write it when the value is...

I've taken a look at this issue and looks like the problem is that Tiled uses a different format for infinite maps. ```xml ... ... .. ``` We can "easily"...

I'm facing a similar issue and I think this can be easily fixed in the case where the entire folder is tracked by dvc. If someone wants to upload a...