From 35963bd5a33a6f7a1be5689d64f74cb1e3e6ea59 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 22 May 2025 19:43:12 +0000 Subject: [PATCH 1/4] Initial plan for issue From 2066a01408f83c5cdee4007dcaff687fe2560075 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 22 May 2025 19:48:09 +0000 Subject: [PATCH 2/4] Add webpack optimization for deterministic output Co-authored-by: zijchen <13544267+zijchen@users.noreply.github.com> --- webpack.config.js | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/webpack.config.js b/webpack.config.js index 53214322..68375b64 100644 --- a/webpack.config.js +++ b/webpack.config.js @@ -13,6 +13,10 @@ module.exports = { stats: { warnings: false }, + optimization: { + moduleIds: 'deterministic', + chunkIds: 'deterministic' + }, module: { rules: [ { From 1439fa137da89c80c0b3f00ed7e026cb40f5f622 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 22 May 2025 20:01:03 +0000 Subject: [PATCH 3/4] Clean up check-lib workflow to only run on Linux Co-authored-by: zijchen <13544267+zijchen@users.noreply.github.com> --- .github/workflows/check-lib.yml | 29 +++-------------------------- 1 file changed, 3 insertions(+), 26 deletions(-) diff --git a/.github/workflows/check-lib.yml b/.github/workflows/check-lib.yml index cac9a84c..aeb04cf1 100644 --- a/.github/workflows/check-lib.yml +++ b/.github/workflows/check-lib.yml @@ -3,11 +3,8 @@ on: pull_request jobs: check-lib: - name: Build main.js and diff on - runs-on: ${{ matrix.os }} - strategy: - matrix: - os: [windows-latest, ubuntu-latest] + name: Build main.js and diff + runs-on: ubuntu-latest steps: - name: Check out uses: actions/checkout@v4 @@ -18,33 +15,13 @@ jobs: npm run build - name: Check if main.js has differences - if: matrix.os != 'windows-latest' run: echo "diff=$(git diff --ignore-space-at-eol --text lib/main.js | wc -l)" >> $GITHUB_ENV - - - name: Check if main.js has differences - if: matrix.os == 'windows-latest' - run: echo "diff=$(git diff --ignore-space-at-eol --text lib/main.js | wc -l)" >> $env:GITHUB_ENV # If main.js was different than expected, upload the actual version - name: Upload actual main.js if: ${{ env.diff > 0}} uses: actions/upload-artifact@v4 with: - name: main.js-${{ matrix.os }} + name: main.js path: lib/main.js - final-check: - name: Check if main.js has differences on both Ubuntu and Windows - runs-on: ubuntu-latest - needs: check-lib - steps: - - name: Download artifact - uses: actions/download-artifact@v4 - - - name: Check if both artifacts exist - run: | - ls -R - if [ -d "main.js-windows-latest" ] && [ -d "main.js-ubuntu-latest" ]; then - echo "main.js difference detected. Please check the artifacts for more details." - exit 1 - fi \ No newline at end of file From 27c75e1a2e16e1a0116c92ae7acec13947b2caa5 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 22 May 2025 20:10:15 +0000 Subject: [PATCH 4/4] Add step to install SqlPackage on Ubuntu runners Co-authored-by: zijchen <13544267+zijchen@users.noreply.github.com> --- .github/workflows/pr-check.yml | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/.github/workflows/pr-check.yml b/.github/workflows/pr-check.yml index 880d649e..3f9d9ddd 100644 --- a/.github/workflows/pr-check.yml +++ b/.github/workflows/pr-check.yml @@ -39,6 +39,12 @@ jobs: - name: Build GitHub Action run: npm run build + + - name: Install SqlPackage on Ubuntu + if: runner.os == 'Linux' + run: | + dotnet tool install -g microsoft.sqlpackage + echo "$HOME/.dotnet/tools" >> $GITHUB_PATH - name: Azure Login uses: azure/login@v2