Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Yaml layer: enable flycheck #7153

Merged
merged 1 commit into from
Oct 14, 2016
Merged

Conversation

rski
Copy link
Contributor

@rski rski commented Sep 20, 2016

No description provided.

(defun yaml/init-yaml-mode ()
"Initialize YAML mode"
(use-package yaml-mode
:defer t
Copy link
Collaborator

Choose a reason for hiding this comment

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

:mode implies :defer, so no need for this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oh, i see, one sec then

@rski rski changed the title Yaml layer: enable flycheck, set defer t Yaml layer: enable flycheck Sep 20, 2016
Copy link
Collaborator

@TheBB TheBB left a comment

Choose a reason for hiding this comment

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

It seems flycheck doesn't syntax check yaml files without external tools. If you could add a documentation section about the available syntax checkers that would be great too.

@robbyoconnor
Copy link
Contributor

@TheBB -- actually all flycheck checkers require external tools. Literally all flycheck does is parse the output from the various tools...

@@ -1,4 +1,4 @@
;;; config.el --- YAML Layer packages File for Spacemacs
;;; config.el --- YAML Layer configuration File for Spacemacs
Copy link
Collaborator

Choose a reason for hiding this comment

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

Thanks for fixing this line. But it should be config 😸

Copy link
Contributor Author

@rski rski Oct 12, 2016

Choose a reason for hiding this comment

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

I can fix it, but I just looked it up from other layers :-)

~/Code/spacemacs/spacemacs on yaml_flycheck ⌚ 20:34:53
$ ag "config file"| wc -l
11

~/Code/spacemacs/spacemacs on  yaml_flycheck ⌚ 20:35:00
$ ag "configuration file"| wc -l
67

Copy link
Collaborator

Choose a reason for hiding this comment

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

Hm... indeed 😸

@d12frosted d12frosted merged commit b27a073 into syl20bnr:develop Oct 14, 2016
@d12frosted
Copy link
Collaborator

Thank you 👍 Cherry-picked into develop, you can safely delete your branch.

@rski rski deleted the yaml_flycheck branch October 14, 2016 11:04
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.

None yet

4 participants