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

💄 Lint #26

Merged
merged 10 commits into from
Dec 6, 2024
Merged

💄 Lint #26

merged 10 commits into from
Dec 6, 2024

Conversation

tolauwae
Copy link
Member

@tolauwae tolauwae commented Dec 6, 2024

  • Enforce consistent indentation
  • Require const declarations for variables that are never reassigned after declared
  • Prefer using the primitive string as a type name, rather than the upper-cased String
  • Disallow unnecessary escape characters
  • Disallow unused variables

- Enforce consistent indentation
- Require `const` declarations for variables that are never reassigned after declared
- Prefer using the primitive string as a type name, rather than the upper-cased String
src/manage/Compiler.ts Fixed Show fixed Hide fixed
src/manage/Compiler.ts Fixed Show fixed Hide fixed
src/manage/Compiler.ts Fixed Show fixed Hide fixed
src/manage/Compiler.ts Fixed Show fixed Hide fixed
src/manage/Compiler.ts Fixed Show fixed Hide fixed
src/sourcemap/SourceMapper.ts Fixed Show fixed Hide fixed
src/sourcemap/SourceMapper.ts Fixed Show fixed Hide fixed
src/sourcemap/SourceMapper.ts Fixed Show fixed Hide fixed
src/sourcemap/SourceMapper.ts Fixed Show fixed Hide fixed
src/sourcemap/SourceMapper.ts Fixed Show fixed Hide fixed
@tolauwae tolauwae added the lint Linter fixes label Dec 6, 2024
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
tolauwae and others added 2 commits December 6, 2024 09:21
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
src/manage/Compiler.ts Fixed Show fixed Hide fixed
tolauwae and others added 6 commits December 6, 2024 10:05
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@tolauwae tolauwae merged commit 05828a8 into main Dec 6, 2024
2 checks passed
@tolauwae tolauwae deleted the lint branch December 6, 2024 09:36
tolauwae added a commit that referenced this pull request Dec 6, 2024
* 💄 Lint

Fixes:
- Enforce consistent indentation
- Require `const` declarations for variables that are never reassigned after declared
- Prefer using the primitive string as a type name, rather than the upper-cased String
- Disallow unnecessary escape characters
- Disallow unused variables

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lint Linter fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant