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

3 participants