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

PycodestyleBear always exits with only one result #2882

Open
areebbeigh opened this issue Mar 3, 2019 · 6 comments · May be fixed by #2904
Open

PycodestyleBear always exits with only one result #2882

areebbeigh opened this issue Mar 3, 2019 · 6 comments · May be fixed by #2904
Assignees

Comments

@areebbeigh
Copy link
Member

@areebbeigh areebbeigh commented Mar 3, 2019

Sample code:

x = 10
y = 5;


print( 'test' ) 

Expected PycodestyleBear output:

bad.py
[   2] y·=·5;
**** PycodestyleBear (E703) [Section: cli | Severity: NORMAL] ****
!    ! E703 statement ends with a semicolon
[    ] *0. Do (N)othing
[    ]  1. (O)pen file
[    ]  2. Add (I)gnore comment
[    ] Enter number (Ctrl-D to exit): 

bad.py
[   5] print(·'test'·)·
**** PycodestyleBear (E201) [Section: cli | Severity: NORMAL] ****
!    ! E201 whitespace after '('
[    ] *0. Do (N)othing
[    ]  1. (O)pen file
[    ]  2. Add (I)gnore comment
[    ] Enter number (Ctrl-D to exit): 

bad.py
[   5] print(·'test'·)·
**** PycodestyleBear (E202) [Section: cli | Severity: NORMAL] ****
!    ! E202 whitespace before ')'
[    ] *0. Do (N)othing
[    ]  1. (O)pen file
[    ]  2. Add (I)gnore comment
[    ] Enter number (Ctrl-D to exit): 

bad.py
[   5] print(·'test'·)·
**** PycodestyleBear (W291) [Section: cli | Severity: NORMAL] ****
!    ! W291 trailing whitespace
[    ] *0. Do (N)othing
[    ]  1. (O)pen file
[    ]  2. Add (I)gnore comment
[    ] Enter number (Ctrl-D to exit):

~ Program exited

Actual output:

bad.py
[   2] y·=·5;
**** PycodestyleBear (E703) [Section: cli | Severity: NORMAL] ****
!    ! E703 statement ends with a semicolon
[    ] *0. Do (N)othing
[    ]  1. (O)pen file
[    ]  2. Add (I)gnore comment
[    ] Enter number (Ctrl-D to exit): 

~ Program exited
@areebbeigh

This comment has been minimized.

Copy link
Member Author

@areebbeigh areebbeigh commented Mar 3, 2019

This is because process_output parses only the first error reported by the linter - introduced in this commit

I would like to be assigned unless this is categorized as a newcomer issue.

@shivank1234

This comment has been minimized.

Copy link

@shivank1234 shivank1234 commented Mar 13, 2019

Hey @areebbeigh Are you working on this?

@areebbeigh

This comment has been minimized.

Copy link
Member Author

@areebbeigh areebbeigh commented Mar 13, 2019

@shivank1234 I need to be assigned first.

@kaixiongg

This comment has been minimized.

Copy link

@kaixiongg kaixiongg commented Apr 8, 2019

Hey I would like to work on this issue if it is considered as a newcomer one.

areebbeigh added a commit to areebbeigh/coala-bears that referenced this issue Apr 9, 2019
This commit modifies process_output to
process all the detected problems instead
of just the first one.

Fixes coala#2882
@areebbeigh areebbeigh mentioned this issue Apr 12, 2019
2 of 2 tasks complete
areebbeigh added a commit to areebbeigh/coala-bears that referenced this issue Apr 19, 2019
This commit modifies process_output to
process all the detected problems instead
of just the first one.

Fixes coala#2882
areebbeigh added a commit to areebbeigh/coala-bears that referenced this issue Apr 19, 2019
This commit modifies process_output to
process all the detected problems instead
of just the first one.

Fixes coala#2882
areebbeigh added a commit to areebbeigh/coala-bears that referenced this issue Apr 19, 2019
This commit modifies process_output to
process all the detected problems instead
of just the first one.

Fixes coala#2882
RuidongLiu added a commit to RuidongLiu/coala-bears that referenced this issue Apr 21, 2019
This commit modifies process_output to
process all the detected problems instead
of just the first one.

Fixes coala#2882
RuidongLiu added a commit to RuidongLiu/coala-bears that referenced this issue Apr 21, 2019
This commit modifies process_output to process
all the detected problems instead of just the first one.

Fixes coala#2882
areebbeigh added a commit to areebbeigh/coala-bears that referenced this issue Jul 22, 2019
This commit modifies process_output to
process all the detected problems instead
of just the first one.

Fixes coala#2882
@Masked-Carrot

This comment has been minimized.

Copy link

@Masked-Carrot Masked-Carrot commented Oct 11, 2019

hello
can I work on this issue

@KVGarg

This comment has been minimized.

Copy link
Contributor

@KVGarg KVGarg commented Oct 13, 2019

hello
can I work on this issue

A PR has already been created for it! Can you please look for a new newcomer issue. Thanks!

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

Successfully merging a pull request may close this issue.

7 participants
You can’t perform that action at this time.