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

Fix grammar in verify tool message #7781

Closed
wants to merge 1 commit into from
Closed

Conversation

@Eilon
Copy link

Eilon commented Dec 18, 2019

No description provided.

@zhhyu

This comment has been minimized.

Copy link
Contributor

zhhyu commented Dec 19, 2019

Thank you for your contributions!

@joelverhagen

This comment has been minimized.

Copy link
Member

joelverhagen commented Dec 19, 2019

Thanks for the change @Eilon. I'll merge this in. We should talk offline about the long term strategy for this verification tool.

@Eilon

This comment has been minimized.

Copy link
Author

Eilon commented Dec 19, 2019

Thanks! Happy to help however I can. I ran this tool as recommended by @karann-msft and it reported happiness to me.

@joelverhagen

This comment has been minimized.

Copy link
Member

joelverhagen commented Feb 13, 2020

This fix was incorporated into #7834.

The updated instructions for the tool are here:
https://github.com/NuGet/NuGetGallery/tree/master/src/VerifyMicrosoftPackage

I have initiated the integration of the official version of the tool into Arcade here:
dotnet/arcade#4838

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.