facebookarchive/atom-ide-ui

Only show lint errors on save?

Open

#72 opened on Oct 4, 2017

View on GitHub
 (2 comments) (15 reactions) (0 assignees)JavaScript (937 stars) (110 forks)batch import
diagnosticsenhancementhelp wanted

Description

Description

I have the linter package disabled, and have the atom-ide-ui package installed along with linter-js-standard. From what I understand, the linter-* packages are compatible with atom-ide-ui, however the errors from the linter-js-standard always show on code change instead of waiting for me to save the file. When using the old linter package, it had an option that allowed you to customize this behavior:

However, I don't believe atom-ide-ui allows you to configure this (except for code formatting).

Expected Behavior

atom-ide-ui should give you an option to allow errors from linter-* packages to only show on save.

Actual Behavior

Errors from linter-* packages always show on change.

Versions

  • Atom: 1.20.1
  • Client OS: macOS Sierra 10.12.6
  • atom-ide-ui: 0.4.0

Additional Details

  • Installed packages (apm ls --installed): Gist

Contributor guide

Tech stack
javascript
Domain
developer experience
Issue type
feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Atom package development
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
35
Research direction
Investigate the settings system in atom ide ui and how it currently handles linter diagnostics. Look at the old linter package's implementation of the 'on save' option for reference. The issue asks for a new configuration option to defer linting until save.
Only show lint errors on save? · facebookarchive/atom-ide-ui#72 | Good First Issue