Skip to content

Create pylint.yml - #56

Open
serapiis wants to merge 2 commits into
ohld:mainfrom
serapiis:main
Open

Create pylint.yml#56
serapiis wants to merge 2 commits into
ohld:mainfrom
serapiis:main

Conversation

@serapiis

@serapiis serapiis commented Aug 9, 2023

Copy link
Copy Markdown

No description provided.

@ohld
ohld requested a review from shamaevnn August 28, 2023 12:42
@shamaevnn

Copy link
Copy Markdown
Collaborator

@serapiis

  1. Likely, that current python code does not meet pylint requirements. You have to run pylint command locally, fix pylint errors and then commit them to the PR.
  2. As far as I know, it's better to have .pylintrc config file for pylint, put it to the repo.
  3. At the moment, we don't have tests. So, I don't think that it's necessary to have django.yml file with tests stage

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants