Skip to content

[AutoRelease] t2-oracledatabase-2024-05-23-80615(can only be merged by SDK owner)#35744

Merged
ChenxiJiang333 merged 4 commits intoAzure:mainfrom
azure-sdk:t2-oracledatabase-2024-05-23-80615
May 30, 2024
Merged

[AutoRelease] t2-oracledatabase-2024-05-23-80615(can only be merged by SDK owner)#35744
ChenxiJiang333 merged 4 commits intoAzure:mainfrom
azure-sdk:t2-oracledatabase-2024-05-23-80615

Conversation

@azure-sdk
Copy link
Copy Markdown
Collaborator

@azure-sdk
Copy link
Copy Markdown
Collaborator Author

@ChenxiJiang333
Copy link
Copy Markdown
Member

/azp run prepare-pipelines

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@ChenxiJiang333
Copy link
Copy Markdown
Member

/check-enforcer override

@ChenxiJiang333 ChenxiJiang333 merged commit 74509c2 into Azure:main May 30, 2024
@azure-sdk azure-sdk deleted the t2-oracledatabase-2024-05-23-80615 branch June 2, 2024 09:12
@@ -0,0 +1,5 @@
# Release History
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@ChenxiJiang333 why is this in the "Oracle" directory? Generally the service directory matches the name of the package so oracledatabase in this case.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for feedback and we will rename the folder soon.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@ChenxiJiang333 why is this in the "Oracle" directory? Generally the service directory matches the name of the package so oracledatabase in this case.

My fault for not checking the output folder config, has made a quick fix on it.

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