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

Allow skip publish DocMS or Github IO for each artifact #11218

Merged
merged 1 commit into from
Sep 15, 2020

Conversation

sima-zhu
Copy link
Contributor

@sima-zhu sima-zhu commented Sep 11, 2020

Added feature of enable/disable DocMs step per artifact level.
Issue:Azure/azure-sdk-tools#922

@sima-zhu sima-zhu self-assigned this Sep 11, 2020
@sima-zhu sima-zhu added the EngSys This issue is impacting the engineering system. label Sep 11, 2020
@mitchdenny
Copy link
Contributor

Description? Why this change, was the previous nested options: element not working?

@sima-zhu
Copy link
Contributor Author

Description? Why this change, was the previous nested options: element not working?

@mitchdenny I have tried several ways to access the nested variable, failed to set and read the value out. There is no necessary to make it so complex, so we decided to flatten the structure.

@sima-zhu sima-zhu merged commit a442db6 into Azure:master Sep 15, 2020
@sima-zhu sima-zhu deleted the add_DocMs branch September 15, 2020 05:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
EngSys This issue is impacting the engineering system.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants