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

Skip copying documentation file when target already exists #75

Merged

Conversation

jnm2
Copy link
Contributor

@jnm2 jnm2 commented Jan 3, 2020

Fixes #74. Verified that File.Copy throws an exception whose (ushort)ex.HResult is 80 due to the target being an existing file.

⚠ This may not be the right (or only) fix needed to close the issue. See issue discussion.

@jnm2 jnm2 force-pushed the documentation_file_already_exists branch from a0920f2 to 5faca5f Compare January 3, 2020 03:12
@jnm2 jnm2 force-pushed the documentation_file_already_exists branch from 5faca5f to fe47e42 Compare January 3, 2020 03:14
@sharwell sharwell merged commit 52022df into tunnelvisionlabs:master Jan 3, 2020
@jnm2 jnm2 deleted the documentation_file_already_exists branch January 3, 2020 19:34
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.

IOException: The file 'obj\Debug\net35\annotated\System.Core.xml' already exists.
2 participants