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

Last boolean attr add #17

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Last boolean attr add #17

wants to merge 2 commits into from

Conversation

MurakamiKennzo
Copy link

close #15

@MurakamiKennzo
Copy link
Author

HI, @rayd I am happy you can reply me , but since I am greener here and can you tell me how to find failed check? and what should I do ? thanks.

@rayd
Copy link
Owner

rayd commented Aug 31, 2018

Hi @MurakamiKennzo, thanks for the fix! Would you mind adding a test to verify the fix?

I think the Travis CI failure is because I've configured it incorrectly. I will try to get this fixed.

@MurakamiKennzo
Copy link
Author

yes, i try to add test similar as your test code in parse-tag.js. but I cannot run npm test. since I am unfamiliar with zuul,. so I cannot fix it and push code.

DeprecationWarning: os.tmpDir() is deprecated. Use os.tmpdir() instead.

it seems node deprecated api bug and configured incorrectly caused. waiting....

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.

Fails for boolean attribute if it is the last attribute on an element
2 participants