-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Typescript with flycheck #2874
Typescript with flycheck #2874
Conversation
Rebase so it's one single commit ;) |
;; URL: https://github.com/syl20bnr/spacemacs | ||
;; | ||
;; This file is not part of GNU Emacs. | ||
;; | ||
;;; License: GPLv3 | ||
|
||
(setq typescript-packages '(tss)) | ||
(setq typescript-private-packages '( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typescript-packages
244d427
to
52b9252
Compare
(use-package tss | ||
:defer t | ||
:mode ("\\.ts\\'" . typescript-mode) | ||
:init (evil-leader/set-key-for-mode 'typescript-mode | ||
:init (evil-leader/set-key-for-mode 'flycheck-mode |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You probably want to use evil-leader/set-key-for-mode 'typescript-mode
to keep this from being bound in other filetypes
6acc830
to
75ebbe8
Compare
add hooks for flycheck while stripping out flymake. As all other packages are written to depend on flycheck, I thought I would strip it out of this package to let us have a consistent interface in all layers. Add author. Cleanup of package.el
1745cee
to
cfe47d6
Compare
Why not contribute the syntax checker to Flycheck? |
this aims to replace the use of flymake with flycheck, giving us a consistent fly* package.