Skip to content
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

Initiate CI run when a Quick Fix executes and triggers a parse. #5113

Open
daFreeMan opened this issue Aug 28, 2019 · 6 comments
Open

Initiate CI run when a Quick Fix executes and triggers a parse. #5113

daFreeMan opened this issue Aug 28, 2019 · 6 comments
Labels
enhancement Feature requests, or enhancements to existing features. Ideas. Anything within the project's scope. feature-inspection-quickfixes

Comments

@daFreeMan
Copy link
Contributor

Justification
I have disabled the Auto-Code Inspection run after a parse. When I'm working through inspections, I don't mind clicking the button to get the CIs to run and the window populated. When I execute a Quick Fix, a parse is triggered, but, since "auto-inspect" is disabled, the CI window is returned empty. I then have to click the button, triggering a parse and the inspection run.

Description
Execution of a Quick Fix should cause the code inspections to run no matter the setting of "Auto Inspect". I'm working on inspections, I don't want to (and shouldn't have to) do any extra clicking or wait for any extra parse runs.

Additional context
2019-08-28 10_50_50-GWX09MOHCWS003 - Remote Desktop Connection - __Remote

@daFreeMan daFreeMan added the enhancement Feature requests, or enhancements to existing features. Ideas. Anything within the project's scope. label Aug 28, 2019
@retailcoder
Copy link
Member

Indeed. Nice catch!

@daFreeMan
Copy link
Contributor Author

An additional bonus would be to run inspections when I open the CI window. I'm not likely to open it without wanting to actually see the inspections. (Another request, or can this be tagged on without too much feature creep?)

@retailcoder
Copy link
Member

Not sure I like a toolwindow that initiates work just by being displayed, without any other user interactions.

@rubberduck203
Copy link
Member

Instead of triggering a whole new parse, why not strategically update the inspections list and parse tree? We know exactly how we’re modifying it, right?

@daFreeMan
Copy link
Contributor Author

And since we're going to strategically update the inspection results, how 'bout we strategically update the CE every time the <enter> key is pressed? We know what changed there...

😁

@retailcoder
Copy link
Member

We know exactly how we’re modifying it, right?

@rubberduck203 as far as inspection results go, we actually don't: it's a clean slate every time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Feature requests, or enhancements to existing features. Ideas. Anything within the project's scope. feature-inspection-quickfixes
Projects
None yet
Development

No branches or pull requests

4 participants