Skip to content

Conversation

nschonni
Copy link
Contributor

@nschonni nschonni commented Nov 7, 2018

Not sure if the pfx file should get added back, but the rest are local user files

@PRMerger13
Copy link
Contributor

@nschonni : Thanks for your contribution! The author, @, has been notified to review your proposed change.

Copy link
Contributor

@mikeblome mikeblome left a comment

Choose a reason for hiding this comment

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

@nschonni Thanks for the PR and for finding these .suo and .pfx file that should not be in the repo. Deleting those files is good. But I'm asking to NOT change the gitignore because we never expect these file types to get checked into a docs repo. Those two examples were the only ones out of hundreds of code snippets. If this ever becomes a problem in the future, of course we can revisit this and perhaps update the file.

@nschonni
Copy link
Contributor Author

nschonni commented Nov 7, 2018

No problem, feel free to fix internally

@nschonni nschonni closed this Nov 7, 2018
@nschonni nschonni deleted the update-gitignore branch November 7, 2018 19:13
@mikeblome
Copy link
Contributor

ok will do thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants