Bash linting about bash HOT 17 CLOSED

I can give this a try and see how it goes after #104 is completed, since it's dependent on that.

from bash.

Forgot about this, I'll see what I can do in the following days.

from bash.

I will look into this again after I have gotten a couple more exercises ported to the track.

from bash.

I'll have another try at this.

from bash.

That looks good and definitely something that will help, I tested it with a sample script and the suggestions were good.

One downside might be with Bats support, as per: koalaman/shellcheck#417, which is not yet merged.

from bash.

That's a great point - I didn't see this before.

If we were set on using shellcheck as a linter, we could omit testing the bats files. Equally, it seems as though there is a good PR attached to the issue, so hopefully it may be merged in the future (or just use the unmerged branch - people seem to think it works pretty well).

Conversely, I have no particular attachment to shellcheck, and as such am happy to take suggestions for any other linters 🙂

from bash.

Linting on the bats file is the test file, and the test file gets seen by the user/learner, and therefore should be as correct as can be.

The solutions are submitted by contributors, and it would be nice to conform to conventions, but this has somewhat less exposure to a learner.

from bash.

Any progress on this one @budmc29 ? 🙂

I imagine there will be a lot of edits with this.

from bash.

Did #148 take care of this?

from bash.

No, this is a different issue.
I don't have time to work on it anymore, so if you can take over if you want.

from bash.

I'll consider this. I'm going to focus on updating the out of date tests with the generator first, so we can get version files for everything.

from bash.

I'll check later today, but I believe shellcheck gets mad if you use [ ] instead of [[ ]].

I think if we implement it we will have to change all of our tests to use [[ ]].

from bash.

I see no issue with that, it's actually an improvement if my understand of how [[ works is correct.

from bash.

Okay, great. I will look into adding linting with shellcheck.

I guess we will need to make other scripts and bin/* follow the same guidelines.

from bash.

A glance through our example solutions suggests that most of them would need changing before they would pass linting with shellcheck. Some of them will need outright replacement

from bash.

I have closed my attempt since I don't have that much time anymore, but if anyone is interested you can see the progress I made on #234

from bash.

I'll close this. If anyone has time to pick it up, feel free to reopen.

from bash.