Skip to content

Commit

Permalink
remove docusaurus machinery from main
Browse files Browse the repository at this point in the history
preserving content

motivation: this clears the way to upgrade our eslint configuration to v9, which should hopefully let us update our custom rule configuration

currently, when attempting to frontport the next js website from v16 to main, we get eslint errors surrounding our custom rules. if we re-implement our custom rules according to the latest eslint configuration, it should make it easier

this change allows us to do that inter alia by removing the eslint typedoc plugin
  • Loading branch information
yaacovCR committed Dec 1, 2024
1 parent 79ad146 commit 63040cc
Show file tree
Hide file tree
Showing 17 changed files with 4,187 additions and 20,265 deletions.
1 change: 0 additions & 1 deletion .eslintignore
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
# Copied from '.gitignore', please keep it in sync.
/.eslintcache
/.docusaurus
/node_modules
/reports
/npmDist
Expand Down
15 changes: 1 addition & 14 deletions .eslintrc.cjs
Original file line number Diff line number Diff line change
Expand Up @@ -420,16 +420,9 @@ module.exports = {
sourceType: 'module',
project: ['tsconfig.json'],
},
plugins: ['@typescript-eslint', 'eslint-plugin-tsdoc'],
plugins: ['@typescript-eslint'],
extends: ['plugin:import/typescript'],
rules: {
//////////////////////////////////////////////////////////////////////////
// `eslint-plugin-tsdoc` rule list based on `v0.3.x`
// https://github.com/microsoft/tsdoc/tree/master/eslint-plugin
//////////////////////////////////////////////////////////////////////////

'tsdoc/syntax': 'error',

//////////////////////////////////////////////////////////////////////////
// `@typescript-eslint/eslint-plugin` rule list based on `v8.4.x`
//////////////////////////////////////////////////////////////////////////
Expand Down Expand Up @@ -750,12 +743,6 @@ module.exports = {
'import/no-commonjs': 'off',
'import/no-nodejs-modules': 'off',
'import/no-extraneous-dependencies': 'off',
// Ignore docusarus related webpack aliases
'n/no-missing-import': 'off',
'import/no-unresolved': [
'error',
{ ignore: ['^@theme', '^@docusaurus', '^@generated'] },
],
},
},
],
Expand Down
29 changes: 0 additions & 29 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -243,32 +243,3 @@ jobs:
with:
name: denoDist
path: ./denoDist

build-website-dist:
name: Build website
runs-on: ubuntu-latest
permissions:
contents: read # for actions/checkout
steps:
- name: Checkout repo
uses: actions/checkout@v4
with:
persist-credentials: false

- name: Setup Node.js
uses: actions/setup-node@v4
with:
cache: npm
node-version-file: '.node-version'

- name: Install Dependencies
run: npm ci --ignore-scripts

- name: Build Docs
run: npm run build:website

- name: Upload denoDist package
uses: actions/upload-artifact@v4
with:
name: websiteDist
path: ./websiteDist
1 change: 0 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@
/diff-npm-package.html
/.eslintcache
/.cspellcache
/.docusaurus
/node_modules
/reports
/npmDist
Expand Down
1 change: 0 additions & 1 deletion .prettierignore
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
# Copied from '.gitignore', please keep it in sync.
/diff-npm-package.html
/.eslintcache
/.docusaurus
/node_modules
/reports
/npmDist
Expand Down
Loading

0 comments on commit 63040cc

Please sign in to comment.