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

LGTM code review alerts #232

Closed
2 tasks done
prjemian opened this issue Aug 16, 2019 · 0 comments · Fixed by #249
Closed
2 tasks done

LGTM code review alerts #232

prjemian opened this issue Aug 16, 2019 · 0 comments · Fixed by #249
Assignees
@prjemian
Labels
maintenance
Milestone
1.1.14

Comments

@prjemian
Copy link
Contributor

prjemian commented Aug 16, 2019 •
edited
Loading

See:
@prjemian prjemian added this to the 1.1.14 milestone Aug 16, 2019
@prjemian prjemian self-assigned this Aug 16, 2019
prjemian added a commit that referenced this issue Sep 3, 2019
@prjemian
MNT #232 remove redundant definition
51b34df
prjemian added a commit that referenced this issue Sep 3, 2019
@prjemian
MNT #232 remove unused import
2d1b71d
prjemian added a commit that referenced this issue Sep 3, 2019
@prjemian
MNT #232 use more informative assertion
7433891
prjemian added a commit that referenced this issue Sep 3, 2019
@prjemian
MNT #232 remove unused import
c2f004d
@prjemian prjemian mentioned this issue Sep 3, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@prjemian prjemian

Labels
maintenance
Projects
None yet
Milestone
1.1.14
Development

Successfully merging a pull request may close this issue.

resolve alerts from LGTM review BCDA-APS/apstools
1 participant
@prjemian