Skip to content

Fish shell linter#1252

Merged
w0rp merged 3 commits into
dense-analysis:masterfrom
nthapaliya:fish-shell-linter
Jan 2, 2018
Merged

Fish shell linter#1252
w0rp merged 3 commits into
dense-analysis:masterfrom
nthapaliya:fish-shell-linter

Conversation

@nthapaliya

Copy link
Copy Markdown
Contributor

Simple linter for fish files, using output from fish -n

The output for fish -n is multiline, so required some hacky parsing.

Tests pass. Docs and readme updated.

I rarely write vim script, so if there are better and more readable ways to achieve what my code does, please let me know.

@w0rp w0rp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, let's go with this.

@w0rp w0rp merged commit 80342b1 into dense-analysis:master Jan 2, 2018
@w0rp

w0rp commented Jan 2, 2018

Copy link
Copy Markdown
Member

Cheers! 🍻

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.

2 participants