Brian Donovan

Results 80 comments of Brian Donovan

Here's [my branch with my changes](https://github.com/eventualbuddha/coffeelint/tree/undefined-rule-wip). It's probably not working in the current state. I went back and looked at my repo, realized I had some changes that had not...

> First, this library is awesome. Thanks! I'm definitely open to suggestions so thanks for filing this issue. Is the thing you're using this for on Github? I'm curious to...

> `throw`ing is an interesting proposal. Recently read this: https://overreacted.io/algebraic-effects-for-the-rest-of-us/. Could allow for some kind of continuation pattern or flexibility and communication not provided by normal functions. That's funny. I...

Actually, I totally forgot that this is what the `match` helper does. You can use it like so: ```ts export const matchRamdaExpr = (node:unknown, ramdaIdent = "R"): [t.MemberExpression, t.Identifier] |...

I took a stab at it in #240. Let me know what you think.

This is a good suggestion. I've been meaning to do something like this for a while, just haven't gotten around to it. I'd be happy to review a PR and/or...

You could do something like this: ```sh # use "M" for modified files, or "??" for untracked, etc git status --porcelain | awk 'match($1, "M") { print $2 }' |...

> But also have some questions on how to do things, is raising an issue a good place for this? Sure, that works. > Regarding this PR though, I noticed...

According to [this](https://docs.github.com/en/actions/managing-workflow-runs/approving-workflow-runs-from-public-forks), there should be an “Approve and run” button but it’s not there. I’ll merge this manually later.

Have you tried it with `--dry-run`?