Skip to content

Remove unused axios dependency - #110

Open
rtibblesbot wants to merge 1 commit into
learningequality:mainfrom
rtibblesbot:issue-109-a18c57
Open

rtibblesbot wants to merge 1 commit into
learningequality:mainfrom
rtibblesbot:issue-109-a18c57

Conversation

@rtibblesbot

@rtibblesbot rtibblesbot commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Removes axios (imported nowhere) and its transitive-only packages, ending its Dependabot PRs.

References

Closes #109. First seen: #74. Lint errors: #100.

Reviewer guidance

AI usage

Used Claude Code to confirm nothing imports axios and to run yarn remove axios. Verified with the full test suite, eslint compared against main, and a frozen-lockfile install.

Deviations from the issue spec

Criterion Why not
yarn lint passes Fails with the same pre-existing errors as main (#100), in files this PR does not touch

Written by rtibblesbot, an LLM-based coding agent.

@rtibblesbot

🟡 Waiting for feedback

Last updated: 2026-10-06 18:27 UTC

Nothing imports it; scripts call GitHub through github.rest.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@rtibblesbot
rtibblesbot marked this pull request as ready for review October 6, 2026 18:23
@rtibblesbot
rtibblesbot requested review from a team and leq-bot October 6, 2026 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove unused axios dependency

1 participant