Skip to content

Spelling public#2260

Merged
vors merged 11 commits intoPowerShell:masterfrom
jsoref:spelling-public
Sep 20, 2016
Merged

Spelling public#2260
vors merged 11 commits intoPowerShell:masterfrom
jsoref:spelling-public

Conversation

@jsoref
Copy link
Copy Markdown
Contributor

@jsoref jsoref commented Sep 14, 2016

These changes are almost entirely composed of changes to public APIs, either classes, methods, or resource identifiers. They're split by directory.

These can definitely be squashed together. Or changes can be split out/dropped if necessary.

@msftclas
Copy link
Copy Markdown

Hi @jsoref, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!

The agreement was validated by Microsoft and real humans are currently evaluating your PR.

TTYL, MSBOT;

@jsoref jsoref force-pushed the spelling-public branch 2 times, most recently from e0711de to 6126552 Compare September 14, 2016 18:27
@vors vors self-assigned this Sep 15, 2016
/// Represents a method that the V2 web service supports to search packages with filtering
/// </summary>
public static readonly string SearchFilterAllersions = "Search()?$orderby=DownloadCount%20desc,Id";
public static readonly string SearchFilterAllVersions = "Search()?$orderby=DownloadCount%20desc,Id";
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@bmanikm @jianyunt is it find to change this API?

<value>PowerShell Script '{0}' Function '{1}' returns null.</value>
<comment>{0} script name, {1} function name</comment>
</data>
<data name="ProvideNameMismatch" xml:space="preserve">
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@jianyunt JFYI, this string is not used anywhere.

@jianyunt
Copy link
Copy Markdown
Contributor

:shipit:

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