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

Question to Mustache Linting #51

Open
Kathrin84 opened this issue Jun 29, 2017 · 3 comments
Open

Question to Mustache Linting #51

Kathrin84 opened this issue Jun 29, 2017 · 3 comments
Labels

Comments

@Kathrin84
Copy link

Hi Mark,
I am quite new to the TravisCI thing and I've got some problems with fully coping the mustache tests.

My current issues:

Maybe you can help me?

Kind regards, Kathrin

@polothy
Copy link
Contributor

polothy commented Jun 29, 2017

I'm not totally up to speed on what the mustache linter does, it's part of core's CI toolset and I'm using it by invoking https://github.com/moodlehq/moodle-local_ci/blob/master/mustache_lint/mustache_lint.php

My guess is that it's coming from parent or child mustache templates or from example rendered content.

The actual HTML validator is this tool, https://validator.github.io/validator/ and I did find a way to ignore messages, but we need to build in support in Moodle's CI tool: https://github.com/validator/validator/wiki/Message-filtering

For the JS, it's using Moodle's eslint (http://eslint.org) integration to lint the JS. If you cannot fix the errors/warnings, you might be able to ignore them, http://eslint.org/docs/user-guide/configuring

@polothy
Copy link
Contributor

polothy commented Jun 29, 2017

I've asked for some help with this issue, but while you wait, you can get your build passing by changing your .travis.yml file to use this instead:

  - moodle-plugin-ci mustache || true

So, it'll always exit with 0, AKA passing. Once things are resolved, I recommend removing || true so you can use Travis to help you discover new errors.

@danpoltawski
Copy link

Sorry about that.

For a second I thought this was caused by https://tracker.moodle.org/browse/MDL-56504

But I see its sort of the opposite problem. What the linter will do is try its best to compile the template including templates it includes and validate the html generated (making use of the example context to try and generate suitable content). Here it looks like you are affected by the bad state of the boost templates?

mchurchward pushed a commit to mchurchward/moodle-plugin-ci that referenced this issue Jan 19, 2021
Don't install grunt globally. Instead run it with npx
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants