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

Look into adding pylint for pull requests and retroactively fix codebase issues #29

Open
ben-ryder opened this issue Sep 20, 2021 · 0 comments · May be fixed by #32
Open

Look into adding pylint for pull requests and retroactively fix codebase issues #29

ben-ryder opened this issue Sep 20, 2021 · 0 comments · May be fixed by #32
Assignees
@ben-ryder
Labels
feature New feature or request
Projects
COE

Comments

@ben-ryder
Copy link
Owner

Describe your feature

  • Add pylint checking for pull requests.
  • Retroactively fix existing issues in repo.

Is your suggestion related to any problems?
n/a

Additional context
n/a
@ben-ryder ben-ryder added the feature New feature or request label Sep 20, 2021
@ben-ryder ben-ryder self-assigned this Sep 20, 2021
ben-ryder added a commit that referenced this issue Sep 20, 2021
@ben-ryder
GH-29: Fixing indent issue in pylint workflow.
1814e68
ben-ryder added a commit that referenced this issue Sep 20, 2021
@ben-ryder
GH-29: Fixing indent issue in pylint workflow.
9befb78
ben-ryder added a commit that referenced this issue Sep 20, 2021
@ben-ryder
GH-29: Fixing version number issue for pylint.
078e6ec
ben-ryder added a commit that referenced this issue Sep 20, 2021
@ben-ryder
GH-29: Updating pylint workflow name.
983149b
ben-ryder added a commit that referenced this issue Sep 20, 2021
@ben-ryder
GH-29: Fixing pylint issues in top level files.
9ddfae9
ben-ryder added a commit that referenced this issue Sep 20, 2021
@ben-ryder
GH-29: Fixing pylint issues in top level files.
a741511
ben-ryder added a commit that referenced this issue Sep 20, 2021
@ben-ryder
GH-29: Adding pylint config and initial progress on pylint fixes for …
ba5c0b6 
…the pygame_gui package. Deleted unused pygame_gui.Cursor element.
@ben-ryder ben-ryder added this to Backlog in COE via automation Sep 23, 2021
@ben-ryder ben-ryder moved this from Backlog to In Progress in COE Sep 23, 2021
ben-ryder added a commit that referenced this issue Sep 23, 2021
@ben-ryder
GH-29: Finishing pylint fixes for pygame_gui package.
83b2c13
@ben-ryder ben-ryder linked a pull request Sep 23, 2021 that will close this issue
Refactoring codebase to fix pylint issues. #32
Draft
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ben-ryder ben-ryder

Labels
feature New feature or request
Projects
COE
  
In Progress
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Refactoring codebase to fix pylint issues. ben-ryder/Conqueror-of-Empires
1 participant
@ben-ryder