Skip to content
This repository has been archived by the owner on Dec 18, 2019. It is now read-only.

downgraded the HTML_CodeSniffer version to fix the own sniffer build #121

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JonasHillerDkd
Copy link

No description provided.

@Hywan Hywan self-assigned this Sep 14, 2018
@Hywan
Copy link
Contributor

Hywan commented Sep 14, 2018

Thanks for the PR!

What's the error you're trying to fix?

@JonasHillerDkd
Copy link
Author

If you want to generate a sniffer, from your own defined HTML_CodeSniffer standard with the command line tool you'll become a an JS error, see #111. This is a result of a basic structure change from the html_CodeSniffer.

old structur of a standard (with the old one the sniffer genration works)
`window.HTMLCS_WCAG2AAA'

new:
'_global.HTMLCS_WCAG2AAA`

so i downgraded the HTML_CodeSniffer

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants