ember-cli / ember-cli/eslint-plugin-ember

use-brace-expansion and spaces

Open
#218 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
263
Forks
214
Avg merge
30m
Merged PRs (30d)
5

Description

In the [docs/rules](https://github.com/ember-cli/eslint-plugin-ember/blob/master/docs/rules/use-brace-expansion.md) it mentions that spaces are not allowed. A developer would also encounter an Ember error upon the code being used in the app (as well as failing tests). However I would argue that this should be caught at the eslint level so it could be fixed before the developer even leaves their editor.

I tried doing this in the existing rule, but ran into a few edge cases where it didn't catch it. When I had the great idea: should this be its own rule or added on to the existing one?

I'm happy to implement it either way, but didn't want to make a judgement call and have it not be inline with the repo. Let me know and I can finish working on it :)

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.