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

Review code to be compliant with pep8 #8

Open
orjanj opened this issue Feb 28, 2021 · 4 comments
Open

Review code to be compliant with pep8 #8

orjanj opened this issue Feb 28, 2021 · 4 comments
Assignees
@Lixona
Labels
good first issue Good for newcomers help wanted Extra attention is needed P: major Priority: major. Below "critical", higher than "default". testing Needs testing
Milestone
Alpha testing

Comments

@orjanj
Copy link
Member

orjanj commented Feb 28, 2021

Review all code commited and make it compliant with pep8
@orjanj orjanj added good first issue Good for newcomers help wanted Extra attention is needed testing Needs testing P: minor Priority: minor. Lowest priority. labels Feb 28, 2021
@orjanj orjanj modified the milestones: Alpha testing, Beta testing Feb 28, 2021
@orjanj orjanj added P: major Priority: major. Below "critical", higher than "default". and removed P: minor Priority: minor. Lowest priority. labels Mar 23, 2021
@orjanj orjanj modified the milestones: Beta testing, Alpha testing Mar 23, 2021
@orjanj
Copy link
Member Author

orjanj commented Mar 23, 2021 •
edited
Loading

Consider to use variables in accordance with Microsoft docs (camel case; must be checked if in accordance with PEP8).

Variable names should either be camel case or snake case.

@create-issue-branch
Copy link

Branch issue-8-Review_code_to_be_compliant_with_pep8 created!

@orjanj orjanj assigned Lixona and unassigned palarssen Apr 10, 2021
@orjanj
Copy link
Member Author

orjanj commented Apr 10, 2021

@Lixona, @palarssen; can you review code in accordance with pep8 and give me feedback asap?

@orjanj orjanj assigned Lixona and unassigned Lixona Apr 10, 2021
@create-issue-branch
Copy link

Branch issue-8-Review_code_to_be_compliant_with_pep8 created!

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

@Lixona Lixona

Labels
good first issue Good for newcomers help wanted Extra attention is needed P: major Priority: major. Below "critical", higher than "default". testing Needs testing
Projects
None yet
Milestone
Alpha testing
Development

No branches or pull requests
3 participants
@orjanj @palarssen @Lixona