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

chore: make eslint chill out #95

Merged
merged 2 commits into from
Mar 9, 2024
Merged

chore: make eslint chill out #95

merged 2 commits into from
Mar 9, 2024

Conversation

helmturner
Copy link
Member

Pull Request Template

  • Fixes:
  • Closes:

Checklist

  • This PR conforms to the guidelines in CONTRIBUTING.md
  • This PR is not a duplicate of a similar PR (how to search issues for similar/duplicate PRs, which this is not
  • This PR targets the dev branch
  • I ran the pre-commit hooks or npm test && npm lint && npm test:e2e and all tests pass

Description

Disables some of the more aggressive lint rules and fixes some Astro lint errors.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Code cleanliness (removal of unnecessary comments or debug logs)

How Has This Been Tested?

I ran the linter and type-checker

Microservices and API Integration

  • If applicable, my changes are compatible with the project's microservices architecture and API gateway integration requirements.

Checklist:

  • My code follows the style guidelines of this project
  • I have made corresponding changes to the documentation
  • I have removed all unnecessary comments and debug log statements
  • I have linked this PR to its corresponding issue

Screenshots (if appropriate):

Please review and suggest any further insights, improvements, or modifications needed.

@zenlex zenlex merged commit 0b498ec into dev Mar 9, 2024
1 check passed
@zenlex zenlex deleted the chore/improve-eslint branch March 9, 2024 18:35
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