Hacker News new | ask | show | jobs
by sharpercoder 3147 days ago
Just tried it out. Smooth sailing! A couple remarks: 1) It would be nice to explain some of the warnings. I had

    myFunction() {
        for (var x in xs) { .. }
        for (var x in xs2) { .. }
    }
it would complain about "Duplicate declaration of variable 'x'". Understandable, but then I'd like to know the javascript context information (apparently x is not in a sub-scope (as in most languages), but in the main function scope). 2) GitHub integration would be nice. A button when looking at source in GitHub which annotates the code with warnings would be quite an improvement over the current view.
1 comments

Thanks for the helpful comment!

1) We will consider adding explanation like "Note that 'x' is declared in function scope. Consider using 'let' declaration if you intended block scope".

2) Yes. Directly showing warnings in the GitHub site would be a nice feature. We will consider it in our roadmap.