-
-
Notifications
You must be signed in to change notification settings - Fork 661
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Run some filters in diagnostics #11220
Conversation
Problem is, this (and 7321f18) also run on As said on slack, there are failures here for find reference requests (for example on 9044 and 9446 display tests), hitting a die + assertion failure while running Checking other failures (damn CI noise with network requests failing again...) Macro failures seem to be mostly the same, with lots of Seems to be hanging locally too if I don't avoid the |
…11226) * Only run filters and save cache on diagnostics, not usage requests * [tests] Update test for 11184
Something else to do here? Test fail seems patched |
…rs_in_diagnostics
Let's see what CI has to say about it |
Nope, still broken here. |
#11229 wasn't a good fix for #11203. I'm not even sure why it does anything at all because that function() {
var future = _Main.Future_Impl_._new();
haxe.Log.trace("much side effect!", {fileName : "source/Main.hx", lineNumber : 13, className : "_Main.Future_Impl_", methodName : "eager"});
} There's a chance that this fix was only incidental, though I don't know how that could happen yet. It also doesn't deal with the situation where |
The story here is that #11229 generated this This took me forever to properly reproduce because I didn't realize that our display tests run without a specified target. This means we have |
* let's see how much breaks * [tests] enable diagnostics tests for 11177 and 11184 * [tests] Update test for 5306 * Don't cache/run filters for find reference/implementation requests (HaxeFoundation#11226) * Only run filters and save cache on diagnostics, not usage requests * [tests] Update test for 11184 * disable test * add VUsedByTyper to avoid bad unused local errors * revert @:compilerGenerated change --------- Co-authored-by: Rudy Ges <k@klabz.org>
* let's see how much breaks * [tests] enable diagnostics tests for 11177 and 11184 * [tests] Update test for 5306 * Don't cache/run filters for find reference/implementation requests (#11226) * Only run filters and save cache on diagnostics, not usage requests * [tests] Update test for 11184 * disable test * add VUsedByTyper to avoid bad unused local errors * revert @:compilerGenerated change --------- Co-authored-by: Rudy Ges <k@klabz.org>
* [display] diagnostics as json rpc (Backport #11412) * [tests] use json rpc diagnostics * [tests] Add test for 11695 * [tests] Update diagnostics tests * Run some filters in diagnostics (#11220) * let's see how much breaks * [tests] enable diagnostics tests for 11177 and 11184 * [tests] Update test for 5306 * Don't cache/run filters for find reference/implementation requests (#11226) * Only run filters and save cache on diagnostics, not usage requests * [tests] Update test for 11184 * disable test * add VUsedByTyper to avoid bad unused local errors * revert @:compilerGenerated change --------- Co-authored-by: Rudy Ges <k@klabz.org> * [display] get rid of TypeloadParse.current_stdin * [display] fix -D display-stdin handling * [display] generalize fileContents behavior to other json rpc display calls * [display] fix range of pattern variables Note: not including texprConverter changes see 160a490 see #7282 * [tests] add test for #7282 * [tests] add test for #7931 * Remove populateCacheFromDisplay config Legacy diagnostics = false, json rpc diagnostics = true * [std] Diagnostics request doc * [tests] Test Json RPC diagnostics with several open files * [diagnostics] fix multi display files (#11722) * [diagnostics] fix json rpc diagnostics display config * [tests] Server tests: do not fail silently when runHaxeJsonCb errors * [tests] add more diagnostics tests * [display] rework multiple display files handling * clean up a bit... * [diagnostics] handle a.b.c.hx case, even if pointless * [diagnostics] do not skip errors during DisplayProcessing.process_display_file * Enable display tests again... * [tests] fix display tests --------- Co-authored-by: Simon Krajewski <simon@haxe.org>
* [display] diagnostics as json rpc (Backport #11412) * [tests] use json rpc diagnostics * [tests] Add test for 11695 * [tests] Update diagnostics tests * Run some filters in diagnostics (#11220) * let's see how much breaks * [tests] enable diagnostics tests for 11177 and 11184 * [tests] Update test for 5306 * Don't cache/run filters for find reference/implementation requests (#11226) * Only run filters and save cache on diagnostics, not usage requests * [tests] Update test for 11184 * disable test * add VUsedByTyper to avoid bad unused local errors * revert @:compilerGenerated change --------- Co-authored-by: Rudy Ges <k@klabz.org> * [display] get rid of TypeloadParse.current_stdin * [display] fix -D display-stdin handling * [display] generalize fileContents behavior to other json rpc display calls * [display] fix range of pattern variables Note: not including texprConverter changes see 160a490 see #7282 * [tests] add test for #7282 * [tests] add test for #7931 * Remove populateCacheFromDisplay config Legacy diagnostics = false, json rpc diagnostics = true * [std] Diagnostics request doc * [tests] Test Json RPC diagnostics with several open files * [diagnostics] fix multi display files (#11722) * [diagnostics] fix json rpc diagnostics display config * [tests] Server tests: do not fail silently when runHaxeJsonCb errors * [tests] add more diagnostics tests * [display] rework multiple display files handling * clean up a bit... * [diagnostics] handle a.b.c.hx case, even if pointless * [diagnostics] do not skip errors during DisplayProcessing.process_display_file * Enable display tests again... * [tests] fix display tests --------- Co-authored-by: Simon Krajewski <simon@haxe.org>
Opening PR so I don't forget, because I kind of did.
Something is still failing which has to be investigated.