DEV: Update CI workflows (#39)

2022-06-17 18:20:09 +02:00 · 2022-06-17 18:20:09 +02:00 · 7b3c12ae15
parent a309e6406d
commit 7b3c12ae15
13 changed files with 1053 additions and 735 deletions
--- a/.eslintrc
+++ b/.eslintrc
@ -1,8 +1,6 @@
 {
  "extends": "eslint-config-discourse",
-  "ignorePatterns": [
-    "javascripts/vendor/*"
-  ],
+  "ignorePatterns": ["javascripts/vendor/*"],
  "globals": {
    "settings": "readonly",
    "themePrefix": "readonly"
--- a/.github/workflows/component-linting.yml
+++ b/.github/workflows/component-linting.yml
@ -0,0 +1,48 @@
+name: Linting
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+
+concurrency:
+  group: plugin-linting-${{ format('{0}-{1}', github.head_ref || github.run_number, github.job) }}
+  cancel-in-progress: true
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v3
+
+      - name: Set up Node.js
+        uses: actions/setup-node@v3
+        with:
+          node-version: 16
+          cache: yarn
+
+      - name: Yarn install
+        run: yarn install
+
+      - name: ESLint
+        if: ${{ always() }}
+        run: yarn eslint --ext .js,.js.es6 --no-error-on-unmatched-pattern {test,javascripts}
+
+      - name: Prettier
+        if: ${{ always() }}
+        shell: bash
+        run: |
+          yarn prettier -v
+          files=$(find javascripts desktop mobile common scss -type f \( -name "*.scss" -or -name "*.js" -or -name "*.es6" \) 2> /dev/null) || true
+          if [ -n "$files" ]; then
+            yarn prettier --list-different $files
+          fi
+          if [ 0 -lt $(find test -type f \( -name "*.js" -or -name "*.es6" \) 2> /dev/null | wc -l) ]; then
+            yarn prettier --list-different "test/**/*.{js,es6}"
+          fi
+
+      - name: Ember template lint
+        if: ${{ always() }}
+        run: yarn ember-template-lint --no-error-on-unmatched-pattern javascripts
--- a/.github/workflows/component-tests.yml
+++ b/.github/workflows/component-tests.yml
@ -6,44 +6,31 @@ on:
      - main
  pull_request:

+concurrency:
+  group: plugin-tests-${{ format('{0}-{1}', github.head_ref || github.run_number, github.job) }}
+  cancel-in-progress: true
+
 jobs:
  build:
    runs-on: ubuntu-latest
    container: discourse/discourse_test:slim-browsers
-    timeout-minutes: 30
+    timeout-minutes: 15

    env:
      DISCOURSE_HOSTNAME: www.example.com
      RUBY_GLOBAL_METHOD_CACHE_SIZE: 131072
      RAILS_ENV: development
-      QUNIT_RAILS_ENV: development
-      PGHOST: postgres
      PGUSER: discourse
      PGPASSWORD: discourse

-    services:
-      postgres:
-        image: postgres:13
-        ports:
-          - 5432:5432
-        env:
-          POSTGRES_USER: discourse
-          POSTGRES_PASSWORD: discourse
-        options: >-
-          --mount type=tmpfs,destination=/var/lib/postgresql/data
-          --health-cmd pg_isready
-          --health-interval 10s
-          --health-timeout 5s
-          --health-retries 5
-
    steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v3
        with:
          repository: discourse/discourse
          fetch-depth: 1

      - name: Install component
-        uses: actions/checkout@v2
+        uses: actions/checkout@v3
        with:
          path: tmp/component
          fetch-depth: 1
@ -64,58 +51,84 @@ jobs:
          sudo -u postgres psql -c "CREATE ROLE $PGUSER LOGIN SUPERUSER PASSWORD '$PGPASSWORD';"

      - name: Bundler cache
-        uses: actions/cache@v2
+        uses: actions/cache@v3
        with:
          path: vendor/bundle
-          key: ${{ runner.os }}-2.7-gem-${{ hashFiles('**/Gemfile.lock') }}
+          key: ${{ runner.os }}-gem-${{ hashFiles('**/Gemfile.lock') }}
          restore-keys: |
-            ${{ runner.os }}-2.7-gem-
+            ${{ runner.os }}-gem-

      - name: Setup gems
        run: |
+          gem install bundler --conservative -v $(awk '/BUNDLED WITH/ { getline; gsub(/ /,""); print $0 }' Gemfile.lock)
          bundle config --local path vendor/bundle
          bundle config --local deployment true
          bundle config --local without development
          bundle install --jobs 4
          bundle clean

+      - name: Lint English locale
+        run: bundle exec ruby script/i18n_lint.rb "tmp/component/locales/en.yml"
+
      - name: Get yarn cache directory
        id: yarn-cache-dir
        run: echo "::set-output name=dir::$(yarn cache dir)"

      - name: Yarn cache
-        uses: actions/cache@v2
+        uses: actions/cache@v3
        id: yarn-cache
        with:
          path: ${{ steps.yarn-cache-dir.outputs.dir }}
-          key: ${{ runner.os }}-${{ matrix.os }}-yarn-${{ hashFiles('**/yarn.lock') }}
+          key: ${{ runner.os }}-yarn-${{ hashFiles('**/yarn.lock') }}
          restore-keys: |
-            ${{ runner.os }}-${{ matrix.os }}-yarn-
+            ${{ runner.os }}-yarn-

      - name: Yarn install
        run: yarn install

-      - name: Migrate database
+      - name: Fetch app state cache
+        uses: actions/cache@v3
+        id: app-cache
+        with:
+          path: tmp/app-cache
+          key: >-
+            ${{ hashFiles('.github/workflows/tests.yml') }}-
+            ${{ hashFiles('db/**/*', 'plugins/**/db/**/*') }}-
+
+      - name: Restore database from cache
+        if: steps.app-cache.outputs.cache-hit == 'true'
+        run: psql -f tmp/app-cache/cache.sql postgres
+
+      - name: Restore uploads from cache
+        if: steps.app-cache.outputs.cache-hit == 'true'
+        run: rm -rf public/uploads && cp -r tmp/app-cache/uploads public/uploads
+
+      - name: Create and migrate database
+        if: steps.app-cache.outputs.cache-hit != 'true'
        run: |
          bin/rake db:create
          bin/rake db:migrate

-      - name: Check qunit existence
+      - name: Dump database for cache
+        if: steps.app-cache.outputs.cache-hit != 'true'
+        run: mkdir -p tmp/app-cache && pg_dumpall > tmp/app-cache/cache.sql
+
+      - name: Dump uploads for cache
+        if: steps.app-cache.outputs.cache-hit != 'true'
+        run: rm -rf tmp/app-cache/uploads && cp -r public/uploads tmp/app-cache/uploads
+
+      - name: Check QUnit existence
        id: check_qunit
        shell: bash
        run: |
-          shopt -s extglob
-          if ls tmp/component/test/**/*.@(js|es6) &> /dev/null; then
+          if [ 0 -lt $(find tmp/component/test -type f \( -name "*.js" -or -name "*.es6" \) 2> /dev/null | wc -l) ]; then
            echo "::set-output name=files_exist::true"
          fi

      - name: Component QUnit
        if: steps.check_qunit.outputs.files_exist == 'true'
        run: |
-          bundle exec rake themes:install -- '--{"${{ github.event.repository.name }}": "tmp/component"}'
-          UNICORN_TIMEOUT=120 bundle exec rake themes:qunit[name,${{ github.event.repository.name }}]
+          THEME_NAME=$(ruby -e 'require "json"; puts JSON.parse(File.read("tmp/component/about.json"))["name"]')
+          bundle exec rake themes:install -- "--{\"$THEME_NAME\": \"tmp/component\"}"
+          QUNIT_EMBER_CLI=1 UNICORN_TIMEOUT=120 bundle exec rake "themes:qunit[name,$THEME_NAME]"
        timeout-minutes: 10
-
-      - name: Lint English locale
-        if: ${{ always() }}
-        run: bundle exec ruby script/i18n_lint.rb "tmp/component/locales/en.yml"
--- a/.github/workflows/theme-linting.yml
+++ b/.github/workflows/theme-linting.yml
@ -1,43 +0,0 @@
-name: Linting
-
-on:
-  push:
-    branches:
-      - main
-  pull_request:
-
-jobs:
-  build:
-    runs-on: ubuntu-latest
-
-    steps:
-      - uses: actions/checkout@v2
-
-      - name: Set up Node.js
-        uses: actions/setup-node@v1
-        with:
-          node-version: 12
-
-      - name: Yarn install
-        run: yarn install
-
-      - name: ESLint
-        if: ${{ always() }}
-        run: yarn eslint --ext .js,.js.es6 --no-error-on-unmatched-pattern {test,javascripts}
-
-      - name: Prettier
-        if: ${{ always() }}
-        shell: bash
-        run: |
-          yarn prettier -v
-          shopt -s extglob
-          if ls @(javascripts|desktop|mobile|common|scss)/**/*.@(scss|js|es6) &> /dev/null; then
-            yarn prettier --list-different "@(javascripts|desktop|mobile|common|scss)/**/*.{scss,js,es6}"
-          fi
-          if ls test/**/*.@(js|es6) &> /dev/null; then
-            yarn prettier --list-different "test/**/*.{js,es6}"
-          fi
-
-      - name: Ember template lint
-        if: ${{ always() }}
-        run: yarn ember-template-lint javascripts
--- a/.gitignore
+++ b/.gitignore
@ -1,3 +1,2 @@
-.discourse-site
-.DS_Store
 node_modules
+.discourse-site
--- a/.prettierrc
+++ b/.prettierrc
@ -0,0 +1 @@
+{}
--- a/README.md
+++ b/README.md
@ -1,4 +1,5 @@
 # DiscoTOC
+
 A Discourse theme component that generates a table of contents for topics with one click

 https://meta.discourse.org/t/discotoc-automatic-table-of-contents/111143
--- a/about.json
+++ b/about.json
@ -1,7 +1,7 @@
 {
  "name": "DiscoTOC",
-  "about_url": "https://meta.discourse.org/t/discotoc-automatic-table-of-contents/111143",
  "component": true,
+  "about_url": "https://meta.discourse.org/t/discotoc-automatic-table-of-contents/111143",
  "license_url": "https://github.com/discourse/DiscoTOC/blob/main/LICENSE",
  "assets": {
    "icons-sprite": "/assets/sprite.svg"
--- a/javascripts/discourse/connectors/below-docs-topic/d-toc-wrapper.hbs
+++ b/javascripts/discourse/connectors/below-docs-topic/d-toc-wrapper.hbs
@ -1 +1 @@
-{{!-- Docs TOC placeholder --}}
+{{! Docs TOC placeholder }}
--- a/javascripts/discourse/connectors/topic-navigation/d-toc-wrapper.hbs
+++ b/javascripts/discourse/connectors/topic-navigation/d-toc-wrapper.hbs
@ -1 +1 @@
-{{!-- TOC placeholder --}}
+{{! TOC placeholder }}
--- a/package.json
+++ b/package.json
@ -5,6 +5,6 @@
  "author": "Discourse",
  "license": "MIT",
  "devDependencies": {
-    "eslint-config-discourse": "^1.1.9"
+    "eslint-config-discourse": "^3.2.0"
  }
 }
--- a/test/acceptance/toc-test.js
+++ b/test/acceptance/toc-test.js
@ -46,8 +46,7 @@ acceptance("DiscoTOC", function (needs) {
              can_wiki: true,
              link_counts: [
                {
-                  url:
-                    "http://127.0.0.1:4200/t/modernizing-the-antiquated-boxing-scoring-system/71/2",
+                  url: "http://127.0.0.1:4200/t/modernizing-the-antiquated-boxing-scoring-system/71/2",
                  internal: true,
                  reflection: false,
                  title: "Modernizing the antiquated boxing scoring system",
--- a/yarn.lock
+++ b/yarn.lock