Skip to content

enhancements #2025

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

Merged
merged 4 commits into from
Apr 10, 2019
Merged

enhancements #2025

merged 4 commits into from
Apr 10, 2019

Conversation

mairaw
Copy link
Contributor

@mairaw mairaw commented Mar 12, 2019

Fixes #1937 plus a few other enhancements

@mairaw mairaw mentioned this pull request Mar 12, 2019
Copy link

@rpetrusha rpetrusha left a comment

Choose a reason for hiding this comment

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

LGTM. Though I'd consider adding information on APIs that use UriKind in the Remarks section. They are:

  • The Uri(Sting,UriKind) constructor (there's only one)
  • Uri.TryCreate(String,UriKind,Uri)
  • Uri.IsWellFormedUriString(UriKind)

@mairaw mairaw added the WIP label Mar 12, 2019
@mairaw
Copy link
Contributor Author

mairaw commented Apr 9, 2019

Like that @rpetrusha? I also need to check after build is done if the @ character on that xref link built successfully or if I need to encode as well.

@mairaw mairaw added waiting-on-feedback Indicates PRs that are waiting for feedback from SMEs before they can be merged and removed WIP labels Apr 9, 2019
Copy link

@rpetrusha rpetrusha left a comment

Choose a reason for hiding this comment

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

This looks good aside from the broken link caused by @, @mairaw.


* <xref:System.Uri.%23ctor(System.String,System.UriKind)?displayProperty=nameWithType>
* <xref:System.Uri.IsWellFormedUriString(System.String,System.UriKind)?displayProperty=nameWithType>
* <xref:M:System.Uri.TryCreate(System.String,System.UriKind,System.Uri@)?displayProperty=nameWithType>

Choose a reason for hiding this comment

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

The link is broken, so I think that this needs to be encoded.

@mairaw mairaw added verify-build-before-merge and removed waiting-on-feedback Indicates PRs that are waiting for feedback from SMEs before they can be merged labels Apr 9, 2019
@mairaw mairaw merged commit f1c3d2a into master Apr 10, 2019
@delete-merged-branch delete-merged-branch bot deleted the mairaw-patch-5 branch April 10, 2019 01:30
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