Skip to content

Commit

Permalink
Merge pull request #326 from letehaha/feat/ui-improvements
Browse files Browse the repository at this point in the history
feat: UI improvements
  • Loading branch information
letehaha authored Dec 31, 2024
2 parents 21e8530 + d9742b1 commit 3e78b87
Show file tree
Hide file tree
Showing 91 changed files with 6,707 additions and 6,421 deletions.
1 change: 1 addition & 0 deletions .eslintignore
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
node_modules
dist
vue.config.js
knip.ts
10 changes: 10 additions & 0 deletions .github/workflows/check-source-code.yml
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,16 @@ jobs:
- name: Build source code
run: npm run build

knip-checkup:
name: No unused files and exports
needs: prepare-dependencies
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: ./.github/actions/prepare-local-env
- name: Run knip
run: npm run knip

# integration-tests:
# name: Integration testing
# needs: prepare-dependencies
Expand Down
55 changes: 55 additions & 0 deletions knip.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,55 @@
// eslint-disable-next-line import/no-self-import
import type { KnipConfig } from "knip";

const config: KnipConfig = {
ignore: [
".eslintrc.js",
"index.d.ts",
"vite.config.js",
"backend/**/**",
"wallaby.js",
// keep it for now, delete when Playwright added
"tests/**/**",
"cypress/**/**",
// keep them all for now
"src/components/lib/**/**",
// for some reason it cannot resolve it
"tsconfig.json",
],
ignoreDependencies: [
// Needed for Storybook
"react-dom",
// Storybook
"@storybook/cli",
"@storybook/theming",
"@storybook/vue3",
"@storybook/addon-docs",
"@storybook/addon-actions",
// needed on CI
"shared-types",
],
// ignoreBinaries: ["eslint"],
rules: {
// Disables "Dubplicate exports" warning. In some components we want to keep
// exporting the component both as the variable and as a "default export"
duplicates: "off",
// enumMembers: "off",
},

/**
* PLUGINS
*
* We define them manually, because Knip cannot find them automatically.
* We need them defined, so Knip won't report of unused dependencies and will
* respect plugins configs.
*/
eslint: {
config: [".eslintrc", ".eslintrc.cjs"],
entry: [".eslintrc.cjs"],
},
/**
* PLUGINS END
*/
};

export default config;
Loading

0 comments on commit 3e78b87

Please sign in to comment.