redwoodjs / redwood

The App Framework for Startups
https://redwoodjs.com
MIT License
17.35k stars 1k forks source link

[Bug]: Linting fails on v8.0.0 javascript projects #11457

Closed Josh-Walker-GM closed 2 months ago

Josh-Walker-GM commented 2 months ago

What's not working?

yarn rw lint fails with the following error:

/Users/jgmw/Development/redwood/rw-test/v8-javascript/api/src/lib/auth.js
  23:31  error  'roles' is defined but never used. Allowed unused args must match /^_/u  no-unused-vars

/Users/jgmw/Development/redwood/rw-test/v8-javascript/web/src/App.jsx
  9:2  error  Parsing error: This experimental syntax requires enabling one of the following parser plugin(s): "jsx", "flow", "typescript". (9:2)

/Users/jgmw/Development/redwood/rw-test/v8-javascript/web/src/Routes.jsx
  14:4  error  Parsing error: This experimental syntax requires enabling one of the following parser plugin(s): "jsx", "flow", "typescript". (14:4)

/Users/jgmw/Development/redwood/rw-test/v8-javascript/web/src/entry.client.jsx
  24:4  error  Parsing error: This experimental syntax requires enabling one of the following parser plugin(s): "jsx", "flow", "typescript". (24:4)

/Users/jgmw/Development/redwood/rw-test/v8-javascript/web/src/pages/FatalErrorPage/FatalErrorPage.jsx
  15:4  error  Parsing error: This experimental syntax requires enabling one of the following parser plugin(s): "jsx", "flow", "typescript". (15:4)

/Users/jgmw/Development/redwood/rw-test/v8-javascript/web/src/pages/NotFoundPage/NotFoundPage.jsx
  2:2  error  Parsing error: This experimental syntax requires enabling one of the following parser plugin(s): "jsx", "flow", "typescript". (2:2)

✖ 6 problems (6 errors, 0 warnings)

This occurs for the default redwood template when you select javascript over typescript.

How do we reproduce the bug?

What's your environment? (If it applies)

System:
    OS: macOS 14.6.1
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.16.0 - /private/var/folders/zb/kc7f0z2d6973pg1p0pjz5nhm0000gn/T/xfs-ba1c093d/node
    Yarn: 4.4.0 - /private/var/folders/zb/kc7f0z2d6973pg1p0pjz5nhm0000gn/T/xfs-ba1c093d/yarn
  Databases:
    SQLite: 3.43.2 - /usr/bin/sqlite3
  Browsers:
    Chrome: 128.0.6613.120
    Safari: 17.6
  npmPackages:
    @redwoodjs/core: 8.0.0 => 8.0.0 
    @redwoodjs/project-config: 8.0.0 => 8.0.0 
  redwood.toml:
    [web]
      title = "Redwood App"
      port = 8910
      apiUrl = "/.redwood/functions" # You can customize graphql and dbauth urls individually too: see https://redwoodjs.com/docs/app-configuration-redwood-toml#api-paths
      includeEnvironmentVariables = [
        # Add any ENV vars that should be available to the web side to this array
        # See https://redwoodjs.com/docs/environment-variables#web
      ]
    [api]
      port = 8911
    [browser]
      open = true
    [notifications]
      versionUpdates = ["latest"]

Are you interested in working on this?

michalgm commented 2 months ago

I can confirm updating to 8.1.0 fixed this issue - thanks for the quick fix!