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

linting errors/warnings notifications unusable in 9.3 #818

Closed
agroszer opened this Issue Nov 6, 2015 · 9 comments

Comments

Projects
None yet
3 participants
@agroszer

agroszer commented Nov 6, 2015

you go to an error with the cursor... no notification...
the other time notifications seem to be fired like a load of bullets... good luck catching the right one

@agroszer

This comment has been minimized.

agroszer commented Nov 6, 2015

at the same time the "Syntax checking status" tab is gone? there at least if was possible to get to the offending line what's impossible now in the nofitications tab :-(

@Naatan Naatan added the Type: Bug label Nov 6, 2015

@Naatan Naatan added this to the 9.3.1 milestone Nov 6, 2015

@Naatan Naatan self-assigned this Nov 6, 2015

@Naatan

This comment has been minimized.

Member

Naatan commented Nov 6, 2015

I have noticed this as well, will look into this.

You can find the same information you found in the Syntax Checking tab by clicking on the Syntax checking icon ad the bottom right of your screen.

@agroszer

This comment has been minimized.

agroszer commented Nov 6, 2015

Little triangle? That has just 3 options: Check, Jump to next, Clear no list

@Naatan

This comment has been minimized.

Member

Naatan commented Nov 6, 2015

Could you provide some sample code where you have syntax errors and the "triangle icon" (doesn't look like a triangle to me?) has no results.

Naatan added a commit that referenced this issue Nov 6, 2015

Linting: Only notify about the first linting error (don't bombard use…
…rs with notifications) - fixes #818

rn=

(integrated from master branch change 9.2.1-674-g845346d by Nathan Rijksen <n.rijksen@gmail.com>)

@Naatan Naatan closed this in 0413dc1 Nov 6, 2015

@agroszer

This comment has been minimized.

agroszer commented Nov 7, 2015

ok, found it:

screenshot - 11072015 - 01 03 35 pm

but it just shows the first 4 problems (that screenshot is of BeautifulSoup.py, having 390 warnings)

@Defman21

This comment has been minimized.

Contributor

Defman21 commented Nov 7, 2015

Having all the 390 items in this popup will break the UI and I don't think there is a way to insert a scrollable box inside this popup.

@agroszer

This comment has been minimized.

agroszer commented Nov 12, 2015

the bombs seem to be gone in
Komodo IDE, version 9.3.1, build 88142, platform linux-x86_64. Built on Mon Nov 9 01:15:29 2015.
but still no solution to show the one under the cursor?

@agroszer

This comment has been minimized.

agroszer commented Nov 12, 2015

well some bombs fire when the linter finishes... sometimes taking away popups...

@Naatan

This comment has been minimized.

Member

Naatan commented Nov 12, 2015

That is a separate issue which is tracked here: #837

I think we are talking about multiple issues here. As the one being mainly addressed here is fixed could you please open a new bug report for your remaining issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment