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

Support a named global namespace #1525

Merged
merged 2 commits into from
Apr 7, 2017

Conversation

jwittner
Copy link
Contributor

@jwittner jwittner commented Apr 5, 2017

Allows specifying an id for the global namespace and includes the global namespace in metadata generation if one is provided.

Resolves #1519

@dnfclas
Copy link

dnfclas commented Apr 5, 2017

@jwittner,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla2.dotnetfoundation.org.

It will cover your contributions to all .NET Foundation-managed open source projects.
Thanks,
.NET Foundation Pull Request Bot

@dnfclas
Copy link

dnfclas commented Apr 5, 2017

@jwittner, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.

Thanks, .NET Foundation Pull Request Bot

@jwittner jwittner force-pushed the dev/globalNamespaceSupport branch from 36bd8e2 to f59d56c Compare April 5, 2017 23:23
{
Logger.LogWarning($"DocFX currently only supports generating metadata with namespace defined. The following types in assembly \"{symbol.MetadataName}\" will have no metadata generated: {string.Join(", ", typeMembers.Select(m => m.MetadataName))}. ");
namespaces = Enumerable.Repeat( symbol.GlobalNamespace, 1);
Copy link
Contributor

Choose a reason for hiding this comment

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

[](start = 47, length = 1)

minor: remove

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed! =)

@superyyrrzz
Copy link
Contributor

:shipit:

@jwittner jwittner force-pushed the dev/globalNamespaceSupport branch from f59d56c to 2cbf75a Compare April 6, 2017 20:08
@superyyrrzz superyyrrzz merged this pull request into dotnet:master Apr 7, 2017
superyyrrzz pushed a commit that referenced this pull request Apr 7, 2017
* Avoid null deref by failing match with against null

* Support global namespaces if a name is supplied
@jwittner
Copy link
Contributor Author

jwittner commented Apr 7, 2017

Thanks a ton for getting this in!

@jwittner jwittner deleted the dev/globalNamespaceSupport branch April 7, 2017 19:46
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.

5 participants