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

Add lint-scope config option #155

Merged
merged 5 commits into from
Jan 19, 2025

Conversation

nobodywasishere
Copy link
Contributor

@nobodywasishere nobodywasishere commented Dec 4, 2024

This tells ameba to either run only on open files or the entire workspace, and is useful for those who want to know what's happening across all files in the workspace instead of open ones.

This also adds a command for linting all files in the workspace, and fixes some minor bugs regarding handling virtual / untitled documents.

@Sija Sija added the enhancement New feature or request label Dec 4, 2024
src/ameba.ts Outdated Show resolved Hide resolved
src/ameba.ts Outdated Show resolved Hide resolved
src/ameba.ts Outdated Show resolved Hide resolved
src/extension.ts Outdated Show resolved Hide resolved
src/ameba.ts Outdated Show resolved Hide resolved
@Sija
Copy link
Member

Sija commented Dec 11, 2024

This PR became unwieldy, please split it into two: a bugfix one, and the new feature additions.

@nobodywasishere
Copy link
Contributor Author

Bug fixes moved to #157

nobodywasishere and others added 5 commits January 3, 2025 16:35
…only on open files or the entire workspace

Modify the ameba execute function to take either a workspace folder or a text document
Co-authored-by: Sijawusz Pur Rahnama <sija@sija.pl>
Co-authored-by: Sijawusz Pur Rahnama <sija@sija.pl>
@nobodywasishere
Copy link
Contributor Author

This is ready for review. I have been using it consistently for the past month and it has worked well for me, from the Crystal compiler to Kagi's codebase, to other smaller projects.

src/ameba.ts Show resolved Hide resolved
Copy link
Member

@veelenga veelenga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@Sija Sija merged commit 2eb75e7 into crystal-ameba:master Jan 19, 2025
2 checks passed
@nobodywasishere nobodywasishere deleted the nobody/lint-scope branch January 19, 2025 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants