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

Re-enable linting on the server #4361

Closed
wants to merge 8 commits into from

Conversation

matt-fidd
Copy link
Contributor

@matt-fidd matt-fidd commented Feb 11, 2025

Re-enables linting on the server. A couple of exceptions to get it through (I'm not sure on the bundler stuff, I can't work out how to remove the extensions from the imports) but can be fixed later.

Requires #4362 to be merged first.

@actual-github-bot actual-github-bot bot changed the title Re-enable linting on the server [WIP] Re-enable linting on the server Feb 11, 2025
Copy link

netlify bot commented Feb 11, 2025

Deploy Preview for actualbudget ready!

Name Link
🔨 Latest commit 003aadc
🔍 Latest deploy log https://app.netlify.com/sites/actualbudget/deploys/67abcf7df6085400084547c5
😎 Deploy Preview https://deploy-preview-4361.demo.actualbudget.org
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@matt-fidd matt-fidd changed the base branch from master to matt/eslint-fix February 11, 2025 22:32
@matt-fidd matt-fidd changed the title [WIP] Re-enable linting on the server Re-enable linting on the server Feb 11, 2025
@matt-fidd matt-fidd deleted the branch actualbudget:matt/eslint-fix February 11, 2025 23:25
@matt-fidd matt-fidd closed this Feb 11, 2025
@matt-fidd
Copy link
Contributor Author

matt-fidd commented Feb 11, 2025

Oops, I expected it to change its base...

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.

1 participant