Skip to content

Test pr 423 #424

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

Closed
wants to merge 6 commits into from
Closed

Test pr 423 #424

wants to merge 6 commits into from

Conversation

gojanpaolo
Copy link
Contributor

@gojanpaolo gojanpaolo commented Mar 24, 2018

WARNING: THIS PR IS MADE JUST TO TEST #423

In summary:

  1. Change UnitsNet code (see "_valueRenamed" in any of the generated quantity classes) which will break serialization library in nuget.
  2. Update JSON code to support the said change
    2a. While in Visual Studio and Debug Mode, the tests still passed which means that the updated JSON code will support the updated UnitsNet code.
  3. Submit this PR.
  4. See Appveyor fail as expected which means that the JSON nuget is not compatible anymore so make sure to release the updated JSON code.

For the serialization test project:
If in debug configuration, use the UnitsNet.Serialization project reference.
If in release configuration, use the latest version of UnitsNet.Serialization as well as UnitsNet in nuget.
Change UnitsNet code (see Mass.g.cs) which will break serialization library in nuget.
Update JsonConverter to support the said change.
@gojanpaolo
Copy link
Contributor Author

Ahhh. Appveyor calls generate-code.bat :(
Need to rethink how to make the breaking change.

Manually change _value to _valueRenamed in the custom codes
@gojanpaolo
Copy link
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant