Skip to content

added cmdlets for logger, properties, git and openIdConnect #2155

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 7 commits into from
Apr 28, 2016
Merged

added cmdlets for logger, properties, git and openIdConnect #2155

merged 7 commits into from
Apr 28, 2016

Conversation

solankisamir
Copy link
Member

  • Added cmdlets for Logger
  • Added cmdlets for Properties
  • Added cmdlets for Git
  • Added cmdlets for OpenIdConnect
  • Fixed issues with Doc generation due to rename of namespace to AzureRm

@azurecla
Copy link

Hi @solankisamir, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!


It looks like you're working at Microsoft (sasolank). If you're full-time, we DON'T require a contribution license agreement.



If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@azuresdkci
Copy link

Can one of the admins verify this patch?

markcowl
markcowl reviewed Apr 26, 2016
View reviewed changes
...esourceManager/ApiManagement/Commands.ApiManagement.ServiceManagement/ApiManagementClient.cs
@@ -1291,5 +1334,410 @@ public void AuthorizationServerRemove(PsApiManagementContext context, string ser
Client.AuthorizationServers.Delete(context.ResourceGroupName, context.ServiceName, serverId, "*");
}
#endregion

#region Loggers
public PsApiManagementLogger LoggerCreate(
Copy link
Member

Choose a reason for hiding this comment

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

No comments for these public methods?

@markcowl
Copy link
Member

@azuresdkci add to whitelist

@solankisamir
review comments around line break, switch parameter, assembly evidence
6285407
@solankisamir
Merge remote-tracking branch 'refs/remotes/Azure/dev' into sasolank_u…
a3a7818 
…pdatepowershell

# Conflicts:

#	src/ResourceManager/ApiManagement/Commands.ApiManagement.ServiceManagement/packages.config

#	src/ResourceManager/ApiManagement/Commands.ApiManagement/packages.config
@solankisamir
Resolve merge issues
945fefe 
merge issues in packages.config
@solankisamir
Add missing BackupRestore files from Websites
a47109a 
The BackupRestore folder got missed as the .gitignore file had an
exclusion clause for Backup*/
@solankisamir
Resolve conflicts during merge with azurecmdfiles.wxi
a0307a7
@markcowl
Copy link
Member

markcowl commented Apr 28, 2016 •
edited
Loading

on demand run here: http://azuresdkci.cloudapp.net/view/1-AzurePowerShell/job/powershell-demand/731/

signing run here: http://azuresdkci.cloudapp.net/view/1-AzurePowerShell/job/powershell-sign/395/

@markcowl
Copy link
Member

@solankisamir merge conflicts. Please update

@solankisamir
Copy link
Member Author

solankisamir commented Apr 28, 2016 •
edited by markcowl
Loading

Merged from latest azure/dev

Powershell On-Demand Run
http://azuresdkci.cloudapp.net/view/1-AzurePowerShell/job/powershell-demand/740/

Powershell Sigin-In Run
http://azuresdkci.cloudapp.net/view/1-AzurePowerShell/job/powershell-sign/404/console

@markcowl markcowl merged commit fea4bb8 into Azure:dev Apr 28, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@solankisamir @azurecla @azuresdkci @markcowl