Skip to content
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

Release 0.59.0 #64

Merged
merged 20 commits into from
Dec 29, 2018
Merged

Release 0.59.0 #64

merged 20 commits into from
Dec 29, 2018

Conversation

michaelcfanning
Copy link
Member

@lgolding FYI.

michaelcfanning and others added 20 commits August 28, 2018 07:14
* Add utilities to validation and todotnet packages

* Update nuget.exe. Add success message for nuspec-driven package creation.

* Update signing scripts
Also:
* Build the `develop` branch in AppVeyor.
Also:
- Use `.CommandLine`, not `.Cli` in the command line tool namespaces.
When you run the ToDotNet command line app, it fails because it can't find the resource stream. Somehow the `.resx` and the `.Resources.designer` files got separated in the project file. Fix the project file to connect the resources back up.
…#60)

#57: When a schema property declares a default value, assign that value to the property in the default constructor. This ensures that the property has the correct default even if you construct the object by hand rather than by deserializing it from JSON.

#58: When a schema property declares a default value, decorate the property with a `System.ComponentModel.DefaultValue` attribute. This avoids your having to specify those attributes in the CodeGenHints.json file, which is error-prone.

Also:
- Fix a bug where the generated doc comments for properties had an extra space: `cref="P: propName`" → `cref="P:propName`".
- Address some hygiene-related IDE messages by marking some properties `readonly`, and by inlining the declarations of some `out` parameters.
@ghost
Copy link

ghost commented Dec 29, 2018

Pro forma +1

@ghost ghost closed this Dec 29, 2018
@ghost ghost reopened this Dec 29, 2018
@ghost
Copy link

ghost commented Dec 29, 2018

Oops, accidentally closed.

@michaelcfanning michaelcfanning merged commit 353fafb into master Dec 29, 2018
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.

3 participants