Skip to content

Conversation

@lmazuel
Copy link
Member

@lmazuel lmazuel commented Feb 15, 2018

@fearthecowboy This is the first time I do a "readme.python.conf" and it works like a charm our of the box.

@marstr @salameer @daschult I still left a "Python" section in the main Readme with a link to the Python readme. I feel it's important, in order for service team to understand they have a Python conf. Can be discussed tomorrow if you want.

@lmazuel lmazuel self-assigned this Feb 15, 2018
@AutorestCI
Copy link

AutorestCI commented Feb 15, 2018

Automation for azure-sdk-for-go

Encountered a Subprocess error: (azure-sdk-for-go)

Command: dep ensure
Finished with return code 127
and output:

/bin/sh: 1: dep: not found

@AutorestCI
Copy link

AutorestCI commented Feb 15, 2018

Automation for azure-sdk-for-python

A PR has been created for you based on this PR content.

Once this PR will be merged, content will be added to your service PR:
Azure/azure-sdk-for-python#1986

@marstr
Copy link
Member

marstr commented Feb 15, 2018

@lmazuel says:
I still left a "Python" section in the main Readme with a link to the Python readme. I feel it's important, in order for service team to understand they have a Python conf.

I think this is a great idea, I'll make sure to emulate this pattern for Go.

@azuresdkciprbot
Copy link

Hi There,

I am the AutoRest Linter Azure bot. I am here to help. My task is to analyze the situation from the AutoRest linter perspective. Please review the below analysis result:

File: specification/hdinsight/resource-manager/readme.md
Before the PR: Warning(s): 0 Error(s): 16
After the PR: Warning(s): 0 Error(s): 16

AutoRest Linter Guidelines | AutoRest Linter Issues | Send feedback

Thanks for your co-operation.

@lmazuel
Copy link
Member Author

lmazuel commented Feb 15, 2018

Finally Travis ended, damn.... Merge it!

@lmazuel lmazuel merged commit 4ed3e62 into Azure:master Feb 15, 2018
@lmazuel lmazuel deleted the hdinsightpyconf branch February 15, 2018 21:01
@lmazuel lmazuel restored the hdinsightpyconf branch February 15, 2018 21:01
@lmazuel lmazuel deleted the hdinsightpyconf branch February 15, 2018 21:03
mccleanp pushed a commit that referenced this pull request Mar 23, 2022
Revert "Marked secrets as not required in deviceupdate.json"
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.

4 participants