Skip to content

Fixes #20150 by updating global-json.md to have cleaner info on match…#21290

Merged
tdykstra merged 5 commits intodotnet:masterfrom
eriawan:patch-1
Nov 2, 2020
Merged

Fixes #20150 by updating global-json.md to have cleaner info on match…#21290
tdykstra merged 5 commits intodotnet:masterfrom
eriawan:patch-1

Conversation

@eriawan
Copy link
Member

@eriawan eriawan commented Oct 31, 2020

Summary

Changes are based on suggestions on #20150 (comment)

Fixes #20150

cc @tdykstra

…ing rules

Fixes #20150 using suggestions from @benmccallum to have cleaner info on matching rules

cc @tdykstra
Co-authored-by: Youssef Victor <31348972+Youssef1313@users.noreply.github.com>
Copy link
Contributor

@tdykstra tdykstra left a comment

Choose a reason for hiding this comment

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

Thanks, looks good -- I made a couple of minor suggestions.

eriawan and others added 2 commits November 2, 2020 23:54
Co-authored-by: Tom Dykstra <tdykstra@microsoft.com>
Co-authored-by: Tom Dykstra <tdykstra@microsoft.com>
@eriawan
Copy link
Member Author

eriawan commented Nov 2, 2020

Thanks @tdykstra 👍

PR updated.

Co-authored-by: Tom Dykstra <tdykstra@microsoft.com>
@eriawan
Copy link
Member Author

eriawan commented Nov 2, 2020

@tdykstra
thanks! PR updated.

@eriawan
Copy link
Member Author

eriawan commented Nov 2, 2020

@tdykstra
Just to remind or to ask, is there anything I have to do?

@tdykstra
Copy link
Contributor

tdykstra commented Nov 2, 2020

No, I'll merge now.

@tdykstra tdykstra self-requested a review November 2, 2020 22:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Specify what the default rollForward value is if not specified

5 participants