Skip to content
This repository has been archived by the owner on Oct 19, 2021. It is now read-only.

no_spaces #556

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

no_spaces #556

wants to merge 2 commits into from

Conversation

JulianWielga
Copy link

Adds no_spaces rule. Related to #135

@swang
Copy link
Collaborator

swang commented Apr 8, 2016

I'll be on the road for the day so I haven't tried this out yet but do the tests actually cover everything? Specifically I'm worried about chained calls such that...

startingChain()
    .hello()
    .world()
    .today((x) -> x + 2)

Specifically .hello is indented with 4 spaces, .world and .today are indented with a tab. My guess(?) is that this may fail because coffeescript slurps the newline\spaces before .hello so it's on the same line as startingChain().

I'll be able to test this more later on, either tonight or after the weekend.

UziTech added a commit to coffeelint/coffeelint that referenced this pull request Jan 14, 2020
@UziTech
Copy link

UziTech commented Jan 14, 2020

I added this to @coffeelint/cli in coffeelint/coffeelint#7. Thanks!

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

Successfully merging this pull request may close these issues.

3 participants