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

Restructure ChangeLogLogic #12769

Closed

Conversation

chidozieononiwu
Copy link
Member

@chidozieononiwu chidozieononiwu commented Jul 28, 2020

Restructure changelog logic to remove reliance on modules and make use of eng\common\scripts\common.ps1
Add Changelog to analyze job as well as release stage to resolve issues raised here

@chidozieononiwu chidozieononiwu force-pushed the RestructureCommonTools branch from 83cfe7d to 5801f59 Compare July 28, 2020 22:27
@chidozieononiwu chidozieononiwu force-pushed the RestructureCommonTools branch 7 times, most recently from 0ec85a9 to 95c6bc8 Compare July 30, 2020 17:56
@praveenkuttappan
Copy link
Member

/azp run python - core - ci

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@@ -28,6 +28,14 @@ stages:
deploy:
steps:
- checkout: self
- script: |
Copy link
Member

Choose a reason for hiding this comment

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

What in our verify-changelog requires the ci_tools to be pip installed first?

Copy link
Member Author

Choose a reason for hiding this comment

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

That's not needed since we are no longer using the previous changelog verification. @praveenkuttappan is there any other why you added this.

openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-python that referenced this pull request Feb 2, 2021
Revert "[DataFactory]Set servicePrincipalId,servicePrincipalKey and tenant to be optional from AzureDataExplorerLinkedServiceTypeProperties (Azure#12755)" (Azure#12769)

This reverts commit baa5ac2.
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.

3 participants