Skip to content

Conversation

rschiefer
Copy link
Contributor

No description provided.

<Platform Condition=" '$(Platform)' == '' ">AnyCPU</Platform>
<ProjectTypeGuids>{11A8DD76-328B-46DF-9F39-F559912D0360};{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}</ProjectTypeGuids>
<ProjectGuid>{3985716e-fa45-4c63-92a7-d656cc46d421}</ProjectGuid>
<ProjectGuid>{9f2a32c7-6fcd-4987-ab43-c0b18d616762}</ProjectGuid>
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see the code generator could be improved to try to keep the guids if they already exist, instead of re-generating every time. Maybe by taking a hash of the project name or similar.

Just an observation, not blocking this PR.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I pushed a fix for deterministic project/solution guids. Please re-generate.

@rschiefer
Copy link
Contributor Author

Think I've got it all in and correct. Apologies if not, still new to UnitsNet and nanoFramework so please bear with me.

@angularsen
Copy link
Owner

Found another issue with the codegen, running codegen locally for me found 1.10.4-preview.4 as latest version instead of preview.11. Fixed in master and merged in here.

@angularsen angularsen merged commit 935c6a6 into angularsen:master May 21, 2021
@angularsen
Copy link
Owner

No need to apologize! This is great work. Most of the problems you ran into were due to the codegen having some minor bugs and inconsistencies that are now fixed as a result of your PR. nanoFramework is still very new to us.

Nuget is hopefully on the way out, within 30 minutes or so. 🎉
https://ci.appveyor.com/project/angularsen/unitsnet/history

Release UnitsNet/4.92.0 · angularsen/UnitsNet

@rschiefer
Copy link
Contributor Author

Awesome, thanks SO MUCH!

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.

2 participants