Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cleanup dependencies #567

Merged
merged 4 commits into from
Jan 23, 2025
Merged

Cleanup dependencies #567

merged 4 commits into from
Jan 23, 2025

Conversation

gbirke
Copy link
Member

@gbirke gbirke commented Jan 21, 2025

Remove prod and dev dependencies that we no longer use

We were using it on the old Privacy page to transfer consent clicks to
Matomo. But 3816fd7 removed that
functionality (we no longer use cookies and therefore no longer need
opt-out or consent) and we can remove the library.
We were using it to format amounts for different locales on the "Use of
Funds" page. Now the page uses pre-formatted numbers from a JSON file.
We were using it for mocking axios, before we introduced jest. Now we're
using different techniques.
This package is a pre-packaged ESLint configuration for Vue projects
created with the `create-vue` tool. Our project predates that tool, was
updated over the years and has its own ESLint configuration that's
incompatible with the configs that the library provides.
@moiikana moiikana merged commit e4c82a0 into main Jan 23, 2025
3 checks passed
@moiikana moiikana deleted the cleanup-dependencies branch January 23, 2025 12:20
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.

2 participants