AtomLinter / linter-eslint-node

ESLint plugin for Atom/Pulsar Linter (v8 and above)
https://web.pulsar-edit.dev/packages/linter-eslint-node
MIT License
4 stars 3 forks source link

Error after fresh install on Ubuntu 22.04 #25

Closed kurttheviking closed 1 year ago

kurttheviking commented 2 years ago

Issue Type

Bug

Issue Description

Just installed Ubuntu 22.04 and reinstalling everything, including Atom. The first package I installed was linter-eslint for v7+. It seems the linter is unable to start and immediately throws getRulesMetaForResults is not a function.

Bug Checklist

n/a
savetheclocktower commented 2 years ago

Does the “Linter Eslint Node: Debug” command not execute at all? The error message you posted is one that would be thrown when the package tries to lint a file, not beforehand. It will try to lint JavaScript files (in ESLint projects) if such a project is restored from a previous session on startup. If it gets that far, the package has initialized successfully.

That error message also suggests a situation where the package thinks it's successfully found a version of ESLint to load, but is wrong. I'm curious how the project in question is structured, and whether ESLint is installed globally or locally.

kurttheviking commented 2 years ago

The error was definitely thrown when it tried to lint a file but I wasn't able to run the debug command. I tried both immediately after a fresh install and after a Atom restart as well as after a full system restart, just to see if that would make a difference. In any case eslint was installed locally (7.22.0) – I was in a hurry to reset everything after the Ubuntu upgrade so I simply downgraded to linter-eslint@9.0.1 and everything seemed to be working as expected again (files linting on save, etc.) So I haven't bothered to go back and try again with linter-eslint-node.

scagood commented 2 years ago

This is related to #27 I think

savetheclocktower commented 1 year ago

This got fixed in #28. We're not publishing to Pulsar's package repo yet, but those who installed straight from GitHub (ppm install AtomLinter/linter-eslint-node) will get the update, I think.