Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: python/memory.python.org
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: main
Choose a base ref
...
head repository: python/memory.python.org
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: frontend-eslint
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 7 commits
  • 28 files changed
  • 3 contributors

Commits on Mar 16, 2026

  1. Add eslint and eslint-config-next as dev dependencies 

    The frontend had an .eslintrc.json config extending next/core-web-vitals
and next/typescript, but the actual eslint packages were never listed
in package.json. This meant `npm run lint` always failed.

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
    @ambv @claude
    ambv and claude committed Mar 16, 2026
    Configuration menu
    Copy the full SHA
    d1327aa View commit details
    Browse the repository at this point in the history

  2. Fix ESLint errors in shared types, hooks, and UI components 

    Remove unused imports. Replace empty interfaces with type aliases.
Fix a const used only as a type. Export DialogClose so it's not
flagged as unused.

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
    @ambv @claude
    ambv and claude committed Mar 16, 2026
    Configuration menu
    Copy the full SHA
    8a02be6 View commit details
    Browse the repository at this point in the history

  3. Fix ESLint errors in admin components 

    Remove unused imports. Drop bindings from catch clauses where the
error variable was unused. Remove dead API_BASE assignments.
Replace require() calls with dynamic import() in QueryConsole.
Escape apostrophes in JSX text.

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
    @ambv @claude
    ambv and claude committed Mar 16, 2026
    Configuration menu
    Copy the full SHA
    801c9bb View commit details
    Browse the repository at this point in the history

  4. Fix ESLint errors in page components 

    Remove unused imports and variables. Drop bindings from catch clauses.
Use const where variables are never reassigned. Remove unused map
callback parameters. Escape apostrophes in JSX text.

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
    @ambv @claude
    ambv and claude committed Mar 16, 2026
    Configuration menu
    Copy the full SHA
    4ea96df View commit details
    Browse the repository at this point in the history

  5. Add ESLint and typecheck to CI, update docs 

    Replace the dummy CI workflow with one that runs ESLint and tsc on
every push and PR. Document how to run both checks via Docker.

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
    @ambv @claude
    ambv and claude committed Mar 16, 2026
    Configuration menu
    Copy the full SHA
    7ac3e43 View commit details
    Browse the repository at this point in the history

  6. Clarify lint execution on CI 

    Co-authored-by: Copilot Autofix powered by AI <[email protected]>
    @ambv
    ambv and Copilot authored Mar 16, 2026
    Configuration menu
    Copy the full SHA
    c551622 View commit details
    Browse the repository at this point in the history

  7. Merge remote-tracking branch 'origin/main' into frontend-eslint 

    # Conflicts:
#	.github/workflows/ci.yml
#	README.md
    @ambv
    ambv committed Mar 16, 2026
    Configuration menu
    Copy the full SHA
    7646d19 View commit details
    Browse the repository at this point in the history
Loading