Merge pull request #55 from microsoft/rzhao271/4146

5.1.11-vscode

Author: rzhao271

.github/workflows/ci.yml

@@ -17,7 +17,7 @@ jobs:
     - uses: actions/checkout@v4
       with:
         persist-credentials: false
-    - run: yarn install
+    - run: npm ci
     - run: scripts/test-electron.sh
   windows:
     runs-on: ${{ matrix.os }}
@@ -28,5 +28,5 @@ jobs:
     - uses: actions/checkout@v4
       with:
         persist-credentials: false
-    - run: yarn install
+    - run: npm ci
     - run: .\scripts\test-electron.bat

.gitignore

@@ -27,5 +27,3 @@ setup.sh
 /build-tmp-napi-v3
 /build-tmp-napi-v6
 *.tgz
-package-lock.json
-yarn.lock

.yarnrc

@@ -17,7 +17,7 @@
         "<!(node -p \"require('node-addon-api').targets\"):node_addon_api_except"
       ],
       "cflags": [
-        "-O2", "-D_FORTIFY_SOURCE=2"
+        "-O2"
       ],
       "msvs_configuration_attributes": {
         "SpectreMitigation": "Spectre"
@@ -28,8 +28,9 @@
             "/guard:cf",
             "/sdl",
             "/W3",
-            "/w34244",
-            "/w34267",
+            "/we4146",
+            "/we4244",
+            "/we4267",
             "/ZH:SHA_256"
           ]
         },