Fix race condition in SyncMessagePort #1197
Workflow file for this run
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| name: CI | |
| defaults: | |
| run: {shell: bash} | |
| env: | |
| PROTOC_VERSION: 3.x | |
| on: | |
| push: | |
| branches: [main, feature.*] | |
| tags: ['**'] | |
| pull_request: | |
| jobs: | |
| static_analysis: | |
| name: Static analysis | |
| runs-on: ubuntu-latest | |
| steps: | |
| - uses: actions/checkout@v4 | |
| - uses: actions/setup-node@v4 | |
| with: | |
| node-version: 'lts/*' | |
| check-latest: true | |
| - name: Check out the language repo | |
| uses: sass/clone-linked-repo@v1 | |
| with: {repo: sass/sass, path: language} | |
| - run: npm install | |
| - name: npm run init | |
| run: | | |
| npm run init -- --skip-compiler --language-path=language $args | |
| - run: npm run check | |
| tests: | |
| name: 'Tests | Node ${{ matrix.node-version }} | ${{ matrix.os }}' | |
| runs-on: ${{ matrix.os }}-latest | |
| strategy: | |
| matrix: | |
| os: [ubuntu, macos, windows] | |
| node-version: ['lts/*', 'lts/-1', 'lts/-2'] | |
| fail-fast: false | |
| steps: | |
| - uses: actions/checkout@v4 | |
| - uses: actions/setup-node@v4 | |
| with: | |
| node-version: ${{ matrix.node-version }} | |
| check-latest: true | |
| - uses: dart-lang/setup-dart@v1 | |
| with: {sdk: stable} | |
| - run: dart --version | |
| - name: Check out Dart Sass | |
| uses: sass/clone-linked-repo@v1 | |
| with: {repo: sass/dart-sass} | |
| - name: Check out the language repo | |
| uses: sass/clone-linked-repo@v1 | |
| with: {repo: sass/sass, path: language} | |
| - run: npm install | |
| - name: npm run init | |
| run: | | |
| npm run init -- --compiler-path=dart-sass --language-path=language $args | |
| - run: npm run test | |
| - run: npm run compile | |
| - run: node test/after-compile-test.mjs | |
| sass_spec: | |
| name: 'JS API Tests | Node ${{ matrix.node_version }} | ${{ matrix.os }}' | |
| runs-on: ${{ matrix.os }}-latest | |
| strategy: | |
| fail-fast: false | |
| matrix: | |
| os: [ubuntu, windows, macos] | |
| node_version: ['lts/*'] | |
| include: | |
| # Include LTS versions on Ubuntu | |
| - os: ubuntu | |
| node_version: lts/-1 | |
| - os: ubuntu | |
| node_version: lts/-2 | |
| steps: | |
| - uses: actions/checkout@v4 | |
| - uses: dart-lang/setup-dart@v1 | |
| with: {sdk: stable} | |
| - uses: actions/setup-node@v4 | |
| with: {node-version: "${{ matrix.node_version }}"} | |
| - name: Check out Dart Sass | |
| uses: sass/clone-linked-repo@v1 | |
| with: {repo: sass/dart-sass} | |
| - name: Check out the language repo | |
| uses: sass/clone-linked-repo@v1 | |
| with: {repo: sass/sass, path: language} | |
| - run: npm install | |
| - name: npm run init | |
| run: | | |
| npm run init -- --compiler-path=dart-sass --language-path=language $args | |
| - name: Check out sass-spec | |
| uses: sass/clone-linked-repo@v1 | |
| with: {repo: sass/sass-spec} | |
| - name: Install sass-spec dependencies | |
| run: npm install | |
| working-directory: sass-spec | |
| - name: Compile | |
| run: | | |
| npm run compile | |
| if [[ "$RUNNER_OS" == "Windows" ]]; then | |
| # Avoid copying the entire Dart Sass build directory on Windows, | |
| # since it may contain symlinks that cp will choke on. | |
| mkdir -p dist/lib/src/vendor/dart-sass/ | |
| cp {`pwd`/,dist/}lib/src/vendor/dart-sass/sass.bat | |
| cp {`pwd`/,dist/}lib/src/vendor/dart-sass/sass.snapshot | |
| else | |
| ln -s {`pwd`/,dist/}lib/src/vendor/dart-sass | |
| fi | |
| - name: Run tests | |
| run: npm run js-api-spec -- --sassPackage .. --sassSassRepo ../language | |
| working-directory: sass-spec | |
| deploy_npm: | |
| name: Deploy npm | |
| runs-on: ubuntu-latest | |
| if: "startsWith(github.ref, 'refs/tags/') && github.repository == 'sass/embedded-host-node'" | |
| needs: [static_analysis, tests, sass_spec] | |
| steps: | |
| - uses: actions/checkout@v4 | |
| - uses: actions/setup-node@v4 | |
| with: | |
| node-version: 'lts/*' | |
| check-latest: true | |
| registry-url: 'https://registry.npmjs.org' | |
| - run: npm install | |
| - name: "Check we're not using a -dev version of the embedded protocol" | |
| run: jq -r '.["protocol-version"]' package.json | grep -qv -- '-dev$' | |
| - name: "Check we're not using a -dev version of the embedded compiler" | |
| run: jq -r '.["compiler-version"]' package.json | grep -qv -- '-dev$' | |
| - name: Publish optional dependencies | |
| env: | |
| NODE_AUTH_TOKEN: '${{ secrets.NPM_TOKEN }}' | |
| run: | | |
| find ./npm -mindepth 1 -maxdepth 1 -print0 | xargs -0 -n 1 -- sh -xc 'npx ts-node ./tool/prepare-optional-release.ts --package=$(basename $1) && npm publish $1' -- | |
| - run: npm publish | |
| env: | |
| NODE_AUTH_TOKEN: '${{ secrets.NPM_TOKEN }}' |