Skip to content

Conversation

@AdamYoblick
Copy link
Contributor

No description provided.

@AdamYoblick AdamYoblick requested a review from dagood November 19, 2019 17:34
Copy link
Member

@dagood dagood left a comment

Choose a reason for hiding this comment

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

Thanks. I think CI will fail though, because this adds a classification for a file that doesn't exist in the current master branch. This commit will need to be rebased onto #187 and pushed. (If you want to keep this change in a non-bot PR, you can also grab the commits from #187 and push them to this PR.)

@AdamYoblick
Copy link
Contributor Author

Thanks Davis, I made the change directly in #187 and pushed it. Going to close this PR without merging.

@AdamYoblick AdamYoblick deleted the update_package_file_list branch November 19, 2019 17:59
@vatsan-madhavan
Copy link
Member

Does Microsoft.VisualBasic.Forms.dll also need to be added ?

/cc @rladuca

@dagood
Copy link
Member

dagood commented Dec 6, 2019

Yeah, but https://github.com/dotnet/winforms/pull/2396/files#diff-6fb3eac8ca40fdf61a2997bd480303fbR9 needs to flow here first so the file actually exists. Currently blocked in WPF: dotnet/wpf#2118.

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