Skip to content

Commit

Permalink
fix!: remove LocaleCode enum
Browse files Browse the repository at this point in the history
This enum often causes tree shaking issues as it’s an Enum Value and it’s only used as a type. Changing in favour of inference based typescript types and an string literal.

Disabled useless auditjs check

BREAKING CHANGE:
LocaleCode enum -> LocaleCode string literal
locales.en -> locales.english
orderedLocales -> orderedLocaleCodes
  • Loading branch information
SimeonC committed Jun 25, 2024
1 parent ad7c730 commit b228bb8
Show file tree
Hide file tree
Showing 25 changed files with 837 additions and 561 deletions.
82 changes: 29 additions & 53 deletions .github/workflows/pull_requests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -57,61 +57,37 @@ jobs:
- name: Run linter
run: npm run lint

audit:
needs: [setup]
runs-on: ubuntu-latest
steps:
- name: Checkout 🛎️
uses: actions/checkout@v2
with:
fetch-depth: 0

- name: Setup Node.js
uses: actions/setup-node@v2
with:
node-version: ${{ needs.setup.outputs.nvmrc }}
cache: npm

- name: Restore npm installs and Lerna setup
run: npm i --legacy-peer-deps

- name: Audit Dependencies
run: npm run audit:ci
env:
OSSI_USERNAME: ${{ secrets.OSSI_USERNAME }}
OSSI_TOKEN: ${{ secrets.OSSI_TOKEN }}

# test:
# needs: [setup]
# runs-on: ubuntu-latest
# steps:
# - name: Checkout 🛎️
# uses: actions/checkout@v2
# with:
# fetch-depth: 0
#
# - name: Setup Node.js
# uses: actions/setup-node@v2
# with:
# node-version: ${{ needs.setup.outputs.nvmrc }}
# cache: npm
#
# - name: Restore npm installs and Lerna setup
# run: npm i --legacy-peer-deps
#
# - name: Run Tests
# run: npm test
#
# - name: Publish Unit Test Results
# uses: EnricoMi/publish-unit-test-result-action@v1
# if: always()
# with:
# files: junit/**/*.xml
# test:
# needs: [setup]
# runs-on: ubuntu-latest
# steps:
# - name: Checkout 🛎️
# uses: actions/checkout@v2
# with:
# fetch-depth: 0
#
# - name: Setup Node.js
# uses: actions/setup-node@v2
# with:
# node-version: ${{ needs.setup.outputs.nvmrc }}
# cache: npm
#
# - name: Restore npm installs and Lerna setup
# run: npm i --legacy-peer-deps
#
# - name: Run Tests
# run: npm test
#
# - name: Publish Unit Test Results
# uses: EnricoMi/publish-unit-test-result-action@v1
# if: always()
# with:
# files: junit/**/*.xml

release:
runs-on: ubuntu-latest
needs: [audit, lint]
# needs: [test, audit, lint]
needs: [lint]
# needs: [test, lint]
steps:
- name: Checkout 🛎️
uses: actions/checkout@v2
Expand All @@ -136,4 +112,4 @@ jobs:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
SLACK_TOKEN: ${{ secrets.SLACK_TOKEN }}
run: npx auto shipit --dry-run
run: npx auto shipit
4 changes: 2 additions & 2 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,8 @@ jobs:
- name: Install 🔧
run: npm ci --legacy-peer-deps

# - name: Run Tests
# run: npm test
# - name: Run Tests
# run: npm test

- name: Create Release ✨
env:
Expand Down
Loading

0 comments on commit b228bb8

Please sign in to comment.