dense-analysis / dense-analysis/ale

Update ALEInfo displayed settings

Open
#4,606 0 comments 1 reaction 1 assignee Claimed by @w0rp View on GitHub
enhancement
Dominant language
Vim Script
Stars
14k
Forks
1.5k
Avg merge
17h 49m
Merged PRs (30d)
1

Description

The `ALEInfo` list of settings to display hasn't been updated in a long time. We should do the following:

1. Set up ALEInfo with a blacklist of setting names to not display, and automatically pull in all ALE global and buffer variables so we don't have to update it as much.
2. Edit the tests for ALEInfo to filter out the setting names so those tests won't break all of the time, except where we explicitly check for the values of settings in output.
3. Add an explicit "always include" list to show names that are lazy-loaded by virtue of autoload files, so we always show settings we know will be set in other files.
4. Redundantly set some default values to display in the results in ALEInfo so we can show what the settings will be should some not-yet-loaded `autoload` files get loaded in later on.

All of the above should make it easier to see what ALE is doing, and discover how to configure it more easily.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.