Merge pull request #94 from heybokeh/cleanup

Cleanup

.github/workflows/main.yml

@@ -1,24 +1,62 @@
-name: tests
+name: CI
 
 on:
   push:
-    branches: [ main ]
+    branches: [main]
   pull_request:
-    branches: [ main ]
+    branches: [main]
 
 jobs:
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout repo
+        uses: actions/checkout@v4
+      - name: Setup Node
+        uses: actions/setup-node@v4
+        with:
+          node-version: 20
+      - name: Install Dependencies
+        uses: pnpm/action-setup@v3
+        with:
+          version: latest
+          run_install: true
+      - name: Lint
+        run: |
+          export NODE_OPTIONS=--max_old_space_size=4096
+          pnpm lint
+
+  test-unit:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout repo
+        uses: actions/checkout@v4
+      - name: Setup Node
+        uses: actions/setup-node@v4
+        with:
+          node-version: 20
+      - name: Install Dependencies
+        uses: pnpm/action-setup@v3
+        with:
+          version: latest
+          run_install: true
+      - name: Unit Tests
+        run: pnpm test
+
   build:
     runs-on: ubuntu-latest
-    strategy:
-      matrix:
-        node-version: [14.x, 16.x]
     steps:
-    - uses: actions/checkout@v2
-    - name: Use Node.js ${{ matrix.node-version }}
-      uses: actions/setup-node@v2
-      with:
-        node-version: ${{ matrix.node-version }}
-        cache: 'npm'
-    - run: npm ci
-    - run: npm run build --if-present
-    - run: npm test
+      - name: Checkout repo
+        uses: actions/checkout@v4
+      - name: Setup Node
+        uses: actions/setup-node@v4
+        with:
+          node-version: 20
+      - name: Install Dependencies
+        uses: pnpm/action-setup@v3
+        with:
+          version: latest
+          run_install: true
+      - name: Build
+        run: pnpm build
+

.gitignore

@@ -2,10 +2,7 @@
 node_modules
 
 # Dist
-/pollen.css
-/index.js
-/index.d.ts
-/utils
+/dist
 
 # Runtime
 coverage

.husky/pre-commit

@@ -1,4 +1,4 @@
 #!/bin/sh
 . "$(dirname "$0")/_/husky.sh"
 
-npm test
+npx lint-staged

.lintstagedrc

@@ -0,0 +1,3 @@
+{
+  "*.{ts}": ["prettier --write", "eslint --quiet"]
+}

.prettierrc

@@ -1,5 +1,4 @@
 {
-  "singleQuote": true,
   "trailingComma": "none",
   "overrides": [
     {

@types/map-obj.d.ts

@@ -1 +1 @@
-declare module 'map-obj';
+declare module "map-obj";

@types/pollen.ts

@@ -1,4 +1,4 @@
-import modules from '../src/modules';
+import modules from "../src/modules";
 type ModuleName = keyof typeof modules;
 
 export type CustomModule = {

CODE_OF_CONDUCT.md

@@ -17,23 +17,23 @@ diverse, inclusive, and healthy community.
 Examples of behavior that contributes to a positive environment for our
 community include:
 
-* Demonstrating empathy and kindness toward other people
-* Being respectful of differing opinions, viewpoints, and experiences
-* Giving and gracefully accepting constructive feedback
-* Accepting responsibility and apologizing to those affected by our mistakes,
+- Demonstrating empathy and kindness toward other people
+- Being respectful of differing opinions, viewpoints, and experiences
+- Giving and gracefully accepting constructive feedback
+- Accepting responsibility and apologizing to those affected by our mistakes,
   and learning from the experience
-* Focusing on what is best not just for us as individuals, but for the
+- Focusing on what is best not just for us as individuals, but for the
   overall community
 
 Examples of unacceptable behavior include:
 
-* The use of sexualized language or imagery, and sexual attention or
+- The use of sexualized language or imagery, and sexual attention or
   advances of any kind
-* Trolling, insulting or derogatory comments, and personal or political attacks
-* Public or private harassment
-* Publishing others' private information, such as a physical or email
+- Trolling, insulting or derogatory comments, and personal or political attacks
+- Public or private harassment
+- Publishing others' private information, such as a physical or email
   address, without their explicit permission
-* Other conduct which could reasonably be considered inappropriate in a
+- Other conduct which could reasonably be considered inappropriate in a
   professional setting
 
 ## Enforcement Responsibilities
@@ -106,7 +106,7 @@ Violating these terms may lead to a permanent ban.
 ### 4. Permanent Ban
 
 **Community Impact**: Demonstrating a pattern of violation of community
-standards, including sustained inappropriate behavior,  harassment of an
+standards, including sustained inappropriate behavior, harassment of an
 individual, or aggression toward or disparagement of classes of individuals.
 
 **Consequence**: A permanent ban from any sort of public interaction within

README.md

@@ -44,20 +44,20 @@ Pollen's design tokens can be used to build any project. They're easy to customi
 
 ```js
 module.exports = (pollen) => ({
-  output: './pollen.css',
+  output: "./pollen.css",
   modules: {
     ...pollen,
     color: {
       ...pollen.colors,
-      bg: 'white',
-      text: 'var(--color-black)'
+      bg: "white",
+      text: "var(--color-black)"
     }
   },
   media: {
-    '(prefers-color-scheme: dark)': {
+    "(prefers-color-scheme: dark)": {
       color: {
-        bg: 'var(--color-black)',
-        text: 'white'
+        bg: "var(--color-black)",
+        text: "white"
       }
     }
   }

eslint.config.js

@@ -0,0 +1,58 @@
+import prettier from "eslint-config-prettier";
+import ts from "typescript-eslint";
+import globals from "globals";
+
+export default [
+  ...ts.configs.recommended,
+  prettier,
+  {
+    ignores: ["**/node_modules/**", "**/*.js"]
+  },
+  {
+    files: ["**/*.ts"],
+    languageOptions: {
+      globals: {
+        ...globals.browser,
+        ...globals.nodeBuiltin
+      }
+    },
+    rules: {
+      curly: "warn",
+      "default-case-last": "warn",
+      eqeqeq: "error",
+      "no-array-constructor": "error",
+      "no-console": "warn",
+      "no-else-return": "warn",
+      "no-multi-assign": "error",
+      "no-nested-ternary": "error",
+      "no-param-reassign": "error",
+      "no-return-assign": "warn",
+      "no-sequences": "error",
+      "no-template-curly-in-string": "warn",
+      "no-throw-literal": "error",
+      "no-undef-init": "warn",
+      "no-unneeded-ternary": "warn",
+      "no-unused-private-class-members": "warn",
+      "no-useless-rename": "warn",
+      "no-var": "error",
+      "prefer-const": "off", // Messes with $props runes
+      "prefer-promise-reject-errors": "warn",
+      "prefer-template": "warn",
+      radix: "warn",
+      "require-yield": "warn",
+      "spaced-comment": "warn",
+      yoda: ["error", "never", { exceptRange: true }],
+      "one-var": ["error", "never"],
+      "@typescript-eslint/no-empty-object-type": "warn",
+      "@typescript-eslint/no-explicit-any": "warn",
+      "@typescript-eslint/no-unused-vars": [
+        "warn",
+        {
+          argsIgnorePattern: "^_",
+          varsIgnorePattern: "^_",
+          caughtErrorsIgnorePattern: "^_"
+        }
+      ]
+    }
+  }
+];