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

Switch syntastic to ALE #649

Merged
merged 1 commit into from
Nov 6, 2024
Merged

Switch syntastic to ALE #649

merged 1 commit into from
Nov 6, 2024

Conversation

choosen
Copy link
Contributor

@choosen choosen commented Nov 5, 2024

Notes from Syntastic:

  1. Deprecation note

This project is no longer maintained. If you need a syntax checking plugin for Vim you might be interested in Syntastic's spiritual succesor, ALE. Although it shares no code with syntastic and it takes a very different approach to design, ALE can be considered a natural evolution of syntastic in terms of goals and functionality. Check it out, you probably won't be disappointed.

  • Updated CHANGELOG
  • Updated README.md (if necessary)

Notes from Syntastic:


1. Deprecation note

This project is no longer maintained. If you need a syntax checking plugin for Vim you might be interested in Syntastic's spiritual succesor, ALE. Although it shares no code with syntastic and it takes a very different approach to design, ALE can be considered a natural evolution of syntastic in terms of goals and functionality. Check it out, you probably won't be disappointed.
@choosen choosen requested a review from cpytel as a code owner November 5, 2024 10:08
@choosen choosen changed the title Switch syntactic to ALE Switch syntastic to ALE Nov 5, 2024
@cpytel
Copy link
Member

cpytel commented Nov 5, 2024

Thanks, is there any reason to keep the mention of Syntastic at all, or should we just remove it entirely at this point?

@choosen
Copy link
Contributor Author

choosen commented Nov 6, 2024

@cpytel I prefer to mention about deprecated one.
When everybody will happy, it can be removed completely.

But I don't know what's your approach to it

@cpytel
Copy link
Member

cpytel commented Nov 6, 2024

Ok, thank you for the contribution.

@cpytel cpytel merged commit 8ce4183 into thoughtbot:main Nov 6, 2024
@choosen choosen deleted the patch-1 branch November 6, 2024 16:14
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