- Enhanched the require rule with an extra required attribute from.bind, this or the from are now required

LittleGnome commented 4 years ago

Enhanched the require rule with an extra required attribute from.bind, this or the from are now required.
Removed unused imports/parameters;
Some other code clean up and update the README.MD;

Make sure you check the following:

[x] you are making a pull request for the develop branch
[ ] unless trivial, a corresponding issue exists for this PR (reference it)
[ ] if this PR fixes the issue, then include fix #
[x] you've ran gulp test and it passes the lint
[x] tests don't fail (except tests in 'failing.spec.ts')

Note: Requests to pull into master from a fork will be rejected.

EisenbergEffect commented 4 years ago

@LittleGnome We actually don't support binding the from attribute of the require element. The require element is much like an ES2015 import and must be statically determined by the template compiler.

LittleGnome commented 4 years ago

But when we use it as followed, it's totally fine, but Aurelia template linter doesn' t accept is. `

aurelia / template-lint

- Enhanched the require rule with an extra required attribute from.bind, this or the from are now required #194