Skip to content
This repository has been archived by the owner on Aug 7, 2023. It is now read-only.

LinterPylint.Linter is deprecated. #50

Closed
s0undt3ch opened this issue Jul 8, 2015 · 9 comments
Closed

LinterPylint.Linter is deprecated. #50

s0undt3ch opened this issue Jul 8, 2015 · 9 comments

Comments

@s0undt3ch
Copy link

AtomLinter v0.X.Y API has been deprecated. Please refer to the Linter docs to update and the latest API: https://github.com/AtomLinter/Linter/wiki/Migrating-to-the-new-API

LinterPylint.Linter (/home/vampas/.atom/packages/linter/lib/linter.coffee:60:4)
new LinterPylint (/home/vampas/.atom/packages/linter-pylint/lib/linter-pylint.coffee:16:4)
Object.lint (/home/vampas/.atom/packages/linter/lib/legacy.coffee:61:21)
<unknown> (/home/vampas/.atom/packages/linter/lib/editor-linter.coffee:68:23)
<unknown> (/home/vampas/.atom/packages/linter/lib/editor-linter.coffee:67:24)
Set.forEach (native)
@JonnyCBB
Copy link

+1

1 similar comment
@wdan
Copy link

wdan commented Jul 12, 2015

+1

@bondgeek
Copy link

+1, guessing this is an AtomLinter problem, not a linter-pylint issue.

@bilderbuchi
Copy link

+1

2 similar comments
@JohnPHogan
Copy link

+1

@rbsolis
Copy link

rbsolis commented Jul 30, 2015

+1

@ma137
Copy link

ma137 commented Aug 3, 2015

Using code from other Atom linter packages, which use linter, I were able to write a pylint based linter which is compatible with the new API. Here is the link: https://github.com/ma0dvl/linter-advpylinter
I had no experience with CoffeeScript, so it would be good if someone would check the code. Maybe a pull request for this repository can be done using my draft.

SpainTrain pushed a commit to SpainTrain/linter-pylint that referenced this issue Aug 13, 2015
* Use latest atom linter API
* Use helpers from atom-linter
* Refactor to use latest linter plugin conventions (main.coffee, etc.)

Closes AtomLinter#52 AtomLinter#53 AtomLinter#51 AtomLinter#50 AtomLinter#49
@SpainTrain
Copy link
Member

Here's a PR #55 that updates to the new API; it should clear up this issue. I tried to keep feature and behavior parity while following the latest atom linter patterns and conventions. Would love for some folks to test it out and shake out any bugs. If you are unfamiliar with installing dev packages from PRs, there are instructions in the PR description. Thanks!

SpainTrain pushed a commit to SpainTrain/linter-pylint that referenced this issue Aug 14, 2015
* Use latest atom linter API
* Use helpers from atom-linter
* Refactor to use latest linter plugin conventions (main.coffee, etc.)

Closes AtomLinter#52 AtomLinter#53 AtomLinter#51 AtomLinter#50 AtomLinter#49
SpainTrain pushed a commit to SpainTrain/linter-pylint that referenced this issue Aug 14, 2015
* Use latest atom linter API
* Use helpers from atom-linter
* Refactor to use latest linter plugin conventions (main.coffee, etc.)

Closes AtomLinter#52 AtomLinter#53 AtomLinter#51 AtomLinter#50 AtomLinter#49
SpainTrain pushed a commit to SpainTrain/linter-pylint that referenced this issue Aug 17, 2015
* Use latest atom linter API
* Use helpers from atom-linter
* Refactor to use latest linter plugin conventions (main.coffee, etc.)

Closes AtomLinter#52 AtomLinter#53 AtomLinter#51 AtomLinter#50 AtomLinter#49
SpainTrain pushed a commit to SpainTrain/linter-pylint that referenced this issue Aug 17, 2015
* Use latest atom linter API
* Use helpers from atom-linter
* Refactor to use latest linter plugin conventions (main.coffee, etc.)

Closes AtomLinter#52 AtomLinter#53 AtomLinter#51 AtomLinter#50 AtomLinter#49
SpainTrain pushed a commit to SpainTrain/linter-pylint that referenced this issue Aug 17, 2015
* Use latest atom linter API
* Use helpers from atom-linter
* Refactor to use latest linter plugin conventions (main.coffee, etc.)

Closes AtomLinter#52 AtomLinter#53 AtomLinter#51 AtomLinter#50 AtomLinter#49
SpainTrain pushed a commit to SpainTrain/linter-pylint that referenced this issue Aug 17, 2015
* Use latest atom linter API
* Use helpers from atom-linter
* Refactor to use latest linter plugin conventions (main.coffee, etc.)

Closes AtomLinter#52 AtomLinter#53 AtomLinter#51 AtomLinter#50 AtomLinter#49
SpainTrain pushed a commit to SpainTrain/linter-pylint that referenced this issue Aug 17, 2015
* Use latest atom linter API
* Use helpers from atom-linter
* Refactor to use latest linter plugin conventions (main.coffee, etc.)

Closes AtomLinter#52 AtomLinter#53 AtomLinter#51 AtomLinter#50 AtomLinter#49
@steelbrain
Copy link
Contributor

Fixed by #55

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests