Skip to content

Remove unused axios dependency #109

Description

@rtibblesbot

❌ This issue is not open for contribution. Visit Contributing guidelines to learn about the contributing process and how to find suitable issues.

Overview

axios is listed in dependencies in package.json, but no file in the repo imports it; the scripts make HTTP calls through github.rest.*. Each axios security release still produces a Dependabot PR to review (e.g. #74).

Complexity: Low
Target branch: main

The Change

  • axios should be removed from package.json and yarn.lock.

Acceptance Criteria

  • package.json has no axios entry.
  • yarn.lock has no axios entry.
  • yarn test and yarn lint pass.

References

AI usage

rtibbles asked for this issue in a review thread on #74 (#74 (comment)); the bot's reply there claimed it was filed before it was. Claude Code checked that nothing imports axios and drafted the issue.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

dependenciesPull requests that update a dependency filejavascriptPull requests that update javascript code

Type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions