{"action":"created","issue":{"url":"https://api.github.com/repos/openstyles/stylus/issues/174","repository_url":"https://api.github.com/repos/openstyles/stylus","labels_url":"https://api.github.com/repos/openstyles/stylus/issues/174/labels{/name}","comments_url":"https://api.github.com/repos/openstyles/stylus/issues/174/comments","events_url":"https://api.github.com/repos/openstyles/stylus/issues/174/events","html_url":"https://github.com/openstyles/stylus/pull/174","id":253462309,"number":174,"title":"re-lint only changed rules","user":{"login":"tophf","id":1310400,"avatar_url":"https://avatars0.githubusercontent.com/u/1310400?v=4","gravatar_id":"","url":"https://api.github.com/users/tophf","html_url":"https://github.com/tophf","followers_url":"https://api.github.com/users/tophf/followers","following_url":"https://api.github.com/users/tophf/following{/other_user}","gists_url":"https://api.github.com/users/tophf/gists{/gist_id}","starred_url":"https://api.github.com/users/tophf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tophf/subscriptions","organizations_url":"https://api.github.com/users/tophf/orgs","repos_url":"https://api.github.com/users/tophf/repos","events_url":"https://api.github.com/users/tophf/events{/privacy}","received_events_url":"https://api.github.com/users/tophf/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":13,"created_at":"2017-08-28T21:27:55Z","updated_at":"2017-08-29T20:43:27Z","closed_at":null,"author_association":"OWNER","pull_request":{"url":"https://api.github.com/repos/openstyles/stylus/pulls/174","html_url":"https://github.com/openstyles/stylus/pull/174","diff_url":"https://github.com/openstyles/stylus/pull/174.diff","patch_url":"https://github.com/openstyles/stylus/pull/174.patch"},"body":"Since both our linters are slow on LARGE code sections, this PR implements partial linting of the changed text. Each change is expanded to its encompassing rule via dumb `}` matching and is extracted along with 2-3 adjacent rules. This extracted fragment is passed to the linter. The new results are merged with the old ones that were outside of the newly checked ranges.\r\n\r\nThe main advantage is that the time spent will be almost the same regardless of a code section size. According to my perf tests on a fast computer, lint-on-typing consumes just a few ms per change. Now the bottleneck is CodeMirror itself.\r\n\r\nI've temporarily changed the delay to update lint report panel to 500ms to demonstrate the increased responsiveness.\r\n\r\n[relint-changes.zip](https://github.com/openstyles/stylus/archive/relint-changes.zip)."},"comment":{"url":"https://api.github.com/repos/openstyles/stylus/issues/comments/325797999","html_url":"https://github.com/openstyles/stylus/pull/174#issuecomment-325797999","issue_url":"https://api.github.com/repos/openstyles/stylus/issues/174","id":325797999,"user":{"login":"Mottie","id":136959,"avatar_url":"https://avatars2.githubusercontent.com/u/136959?v=4","gravatar_id":"","url":"https://api.github.com/users/Mottie","html_url":"https://github.com/Mottie","followers_url":"https://api.github.com/users/Mottie/followers","following_url":"https://api.github.com/users/Mottie/following{/other_user}","gists_url":"https://api.github.com/users/Mottie/gists{/gist_id}","starred_url":"https://api.github.com/users/Mottie/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Mottie/subscriptions","organizations_url":"https://api.github.com/users/Mottie/orgs","repos_url":"https://api.github.com/users/Mottie/repos","events_url":"https://api.github.com/users/Mottie/events{/privacy}","received_events_url":"https://api.github.com/users/Mottie/received_events","type":"User","site_admin":false},"created_at":"2017-08-29T20:43:27Z","updated_at":"2017-08-29T20:43:27Z","author_association":"MEMBER","body":">  Doesn't it allow adding additional tweaks to each rule besides the common on/off and severity? If yes, it'd be tricky to implement, but still possible I guess.\r\n\r\nYes, like the `\"declaration-block-no-duplicate-properties\"` rule... JSON would be the source of the code that builds the form, so it *shouldn't be too difficult* (famous last words). I'll take a look when I get some time later this evening."}}
{"actor":{"display_login":"Mottie"}}