New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merge YAXLib.Redux v2 #122
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What happend to the snk? Why is that one changed?
YAXLib/YAXLib.csproj
Outdated
</PropertyGroup> | ||
<PropertyGroup Condition="'$(Configuration)|$(TargetFramework)|$(Platform)'=='Release|netstandard2.0|AnyCPU'"> | ||
<DefineConstants>TRACE</DefineConstants> | ||
<PackageTags>XML Serialization Serializer Serialize .NET Core</PackageTags> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this on purpose?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PackageTags
? These are the NuGet tags.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes as requested by @304NotModified
Sorry for the conflicts. Did not expect that after merging the other PR. |
adac923
to
043dd28
Compare
7c2a654
to
2c9fe2b
Compare
Please review the rebased pr. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2c9fe2b
to
b9128a2
Compare
Thanks! Will try to review this weekend |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks great work!
@axunonb agree with merge of this one? |
Rework of #120 excluding newline normalization and coding style changes.