search

MicrosoftDocs / CordovaDocs

content repo for Apache Cordova on docs.microsoft.com
https://docs.microsoft.com/en-us/visualstudio/cross-platform/tools-for-cordova/
Creative Commons Attribution 4.0 International
14 stars 32 forks source link

Update best-practices.md #23

Closed kencenerelli closed 6 years ago

kencenerelli commented 6 years ago

Minor typos.

conceptdev commented 6 years ago

@kencenerelli thanks for the PR! we are currently in the process of refactoring the Cordova docs to include both VS2015 and VS2017, with a cool version-switcher. We don't want to miss your updates tho' - I'll be sure to make sure they're incorporated into the content.

kencenerelli commented 6 years ago

Thanks Craig for your response.

conceptdev commented 6 years ago

@kencenerelli thanks again for the fixes. we've just pushed our updated content, which includes VS2015 and VS2017 content side-by-side (see the new 'version switcher' above the navigation tree).

Unfortunately this created a merge conflict for your PR. To prevent you having to re-do it, I've made your changes directly in the new folder structure, so I'm closing this PR.

If you have further questions, please continue to respond to this thread, or open an issue. We would welcome future contributions.

kencenerelli commented 6 years ago

Thanks Craig for taking my edits into consideration.