Update to use CFS feed (#4078)

* update hosted-git-info version * add npmrc and update yarn lock * we were hitting an issue with eslint getting updated, I think it's with eslint >= 8.31.0 * attempt to fix cfs and get from our feed without updating versions * try without hosted-git-info resolution * revert to base * updating yarnpkg to our feed * add comment to npmrc * Update CONTRIBUTING.md * Update .npmrc Co-authored-by: Garrett Campbell <[email protected]> --------- Co-authored-by: Garrett Campbell <[email protected]> Co-authored-by: Garrett Campbell <[email protected]>
microsoft · Sep 19, 2024 · 0df87e4 · 0df87e4
1 parent 202da03
commit 0df87e4
Show file tree

Hide file tree

Showing 3 changed files with 1,077 additions and 1,065 deletions.
diff --git a/.npmrc b/.npmrc
@@ -0,0 +1,4 @@
+# [OSS Contributors] Delete this .npmrc file to use the npmjs registry instead of the Microsoft Azure Artifacts feed for local development
+
+registry=https://pkgs.dev.azure.com/azure-public/VisualCpp/_packaging/cpp_PublicPackages/npm/registry/                  
+always-auth=true
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
@@ -34,3 +34,11 @@ Follow the [Typescript Coding guidelines](https://github.com/Microsoft/TypeScrip
 ### Changelog
 
 Please update the CHANGELOG.md file as part of your pull request. Follow the style within that file and give yourself credit for the changes you make.
+
+### Adding/Updating package.json dependencies
+
+We maintain a public Azure Artifacts feed that we point the package manager to in the .npmrc file.
+
+Because of this, if you want to add/update a dependency version in package.json, intervention from our team will be required to successfully merge PRs that have dependency updates. Please ping our team in the PR.
+
+For local development before attempting to merge a PR, you can delete the .npmrc file and use the default NPM feed directly. Please do not push any subsequent changes to the yarn.lock file.