catull
catull
> Hey @catull, > > I checked earlier with our product team and they said they would also be expecting something like: > > ``` > > ``` > >...
> In my opinion `` looks nicer. > > Thanks, Daniel. For which case ? I corrected my suggestion. Are you referring to the case of a SQL NULL value...
> > In my opinion `` looks nicer. > > Thanks, Daniel. > > For which case ? I corrected my suggestion. > > Are you referring to the case...
Dear @filipelautert I made the suggested changes related to explicitly export `null` values in XML format. Can you review the changes, please ?
> Hi @catull, > > I have a few thing I have observed on the current state of this PR: > > 1. I think temporarily we should ignore `GenerateSqlChangeLogPostgresIntegrationTest`...
> Hey @catull, > > Could you please review the changes made on the serializer, I think they are not yet fixing these cases. For example what has been added...
@MalloD12 Can you please review the changes ? Thanks.
> Hi @catull, > > I triggered the tests for this PR and am still seeing some failures as you can see in the below screenshot: > > > I...
@MalloD12 Fixed the 2 cases in `GenerateYamlChangeLogPostgresIntegrationTest` as well as the third case reported against `GenerateJsonChangeLogPostgresIntegrationTest`. Have a pleasant first week-end of the year.
@MalloD12 I do not see a failure/exception with `GenerateChangeLogEmptyIntegrationTest`.