Skip to content

Conversation

@xendk
Copy link
Collaborator

@xendk xendk commented Mar 3, 2014

Properly chain into the checker chain for js/css modes. Also check info
files. Closes #41.

Take a look @arnested ...

Properly chain into the checker chain for js/css modes. Also check info
files. Closes #41.
@arnested
Copy link
Owner

arnested commented Mar 4, 2014

Looks fine although I'm not familiar with details of flycheck.el (yet?).

Merged and thank you.

arnested added a commit that referenced this pull request Mar 4, 2014
Use flycheck-add-next-checker.
@arnested arnested merged commit 7402b9e into develop Mar 4, 2014
@arnested arnested deleted the flycheck-update branch March 4, 2014 20:58
@dhaley
Copy link

dhaley commented Mar 10, 2014

@xendk, thanks for the flycheck 'drupal/phpcs' checker. It's working great for me under drupal-mode - latest 'development' HEAD.

I was able to take the hardcoded:

'Drupal', ) ?>

out of my CodeSniffer.conf so I can now use other flycheck phpcs checkers on non-Drupal php files.

@arnested
Copy link
Owner

Good to know, @dhaley.

I just released it as version 0.5.0.

@xendk xendk restored the flycheck-update branch September 1, 2014 08:31
@xendk xendk deleted the flycheck-update branch September 1, 2014 08:31
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.

Update flycheck support

4 participants