Skip to content

Update the script to include symbolExclusion file#7952

Merged
epananth merged 7 commits into
dotnet:mainfrom
epananth:symb-validation
Oct 18, 2021
Merged

Update the script to include symbolExclusion file#7952
epananth merged 7 commits into
dotnet:mainfrom
epananth:symb-validation

Conversation

@epananth

Copy link
Copy Markdown
Member

To double check:

In symbol-validation we need a param to SymbolExclusionfile. This will help us exclude the files which were not published and just skip them.

Comment thread eng/common/post-build/symbols-validation.ps1 Outdated
Comment thread eng/common/post-build/symbols-validation.ps1 Outdated
Comment thread eng/common/post-build/symbols-validation.ps1 Outdated
Comment thread eng/common/post-build/symbols-validation.ps1 Outdated

@michellemcdaniel michellemcdaniel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. Let's make sure this is fully tested in the staging pipeline, but I'm happy

@epananth

Copy link
Copy Markdown
Member Author

Yep running some dotnet-release build.. Once that passes I will merge this..

Comment thread eng/common/post-build/symbols-validation.ps1 Outdated
@epananth epananth added the auto-merge Automatically merge PR once CI passes. label Oct 18, 2021
@ghost

ghost commented Oct 18, 2021

Copy link
Copy Markdown

Hello @epananth!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@epananth
epananth merged commit 188c7b1 into dotnet:main Oct 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge Automatically merge PR once CI passes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants