node-red / nrlint

Node-RED Flow Linter
Apache License 2.0
36 stars 15 forks source link

Restructuring plugins #3

Closed k-toumura closed 3 years ago

k-toumura commented 4 years ago

Proposed changes

Checklist

k-toumura commented 3 years ago

This PR includes a commit of PR #2. To avoid conflict, I've closed PR #2, and copied the comment to below:


Proposed changes

This PR enables an Editor-integrated linter function so that flow developer can validate their code on the fly.

remaining issues


knolleary commented 3 years ago

Given the size of this PR, I'm going to merge it as-is without doing a full review.

Following our call this week, I plan to spend time in the next few days going through this code base to see how you've approached the plugin loading. That will help inform what is needed for the main plugin design for Node-RED.

We may need to make some changes here based on the outcome of that design - but will be easier to do with follow-on PRs rather than try to manage it all under this one.