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.

Sign up for GitHub

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

Jump to bottom

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
@areebbeigh
Labels
difficulty/low type/bug

Comments

@areebbeigh
Copy link
Member

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
Copy link
Member Author

areebbeigh commented Mar 3, 2019 •
edited
Loading

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.

@areebbeigh areebbeigh mentioned this issue Mar 3, 2019
Open
@shivank1234
Copy link

Hey @areebbeigh Are you working on this?

@areebbeigh
Copy link
Member Author

@shivank1234 I need to be assigned first.

@kaixiongg
Copy link

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
@areebbeigh
Process whole output
71ee34c 
This commit modifies process_output to
process all the detected problems instead
of just the first one.

Fixes coala#2882
@areebbeigh areebbeigh linked a pull request Apr 12, 2019 that will close this issue
Open
2 tasks
areebbeigh added a commit to areebbeigh/coala-bears that referenced this issue Apr 19, 2019
@areebbeigh
Process whole output
2f60167 
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
@areebbeigh
Process whole output
de7d25c 
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
@areebbeigh
Process whole output
8bd8fff 
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
@RuidongLiu
PycodestyleBear.py: Find all bad codestyles
49dd88e 
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
@RuidongLiu
PycodestyleBear.py: Find all bad codestyles
8d69ec1 
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
@areebbeigh
Process whole output
7984739 
This commit modifies process_output to
process all the detected problems instead
of just the first one.

Fixes coala#2882
@MaskedCarrot
Copy link

hello
can I work on this issue

@KVGarg
Copy link
Contributor

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
Assignees

@areebbeigh areebbeigh

Labels
difficulty/low type/bug
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

PycodestyleBear: Process whole output areebbeigh/coala-bears
7 participants
@areebbeigh @gitmate-bot @li-boxuan @KVGarg @shivank1234 @kaixiongg @MaskedCarrot