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

Upgrade to SublimeLinter4 #2

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

siderealex
Copy link

@siderealex siderealex commented Apr 4, 2020

Hello,

SublimeLinter4 has been stable for > 2 years now, with SublimeLinter3 being deprecated. I wanted to use this plugin with SublimeLinter4, so went about upgrading it.

The major changes are:

  • Use a custom config setting instead of the default args, which felt ambiguous due to the need for both executable args as well as program args.
  • Use the flake8/pycodestyle recommended setup.cfg for project linting configuration, instead of the deprecated .sublimelinterrc
  • Add some additional text to README.md for enhanced clarity

Otherwise I attempted to keep the functionality the same, namely implementing custom logic to find scalastyle-config.xml in parent directories, since the previously built-in config_file functionality appears to be removed from SublimeLinter 4.

Thanks for having a look.

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