-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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.
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
Develop #1054
base: master
Are you sure you want to change the base?
Develop #1054
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Several changes were requested.
requirements.txt
Outdated
flake8==5.0.4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do not change the requirements.txt
, this file should be absent in the pull request.
requirements.txt
Outdated
flake8-variables-names==0.0.5 | ||
pep8-naming==0.13.2 | ||
pytest==7.1.3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Exclude the content of the __pycache__
directory and the directory itself from the PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove pycache from repo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
still not fixed
Якщо неправильно, то підскажіть де може знаходитися pycache |
Ніби все правильно зробив |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Leave only app/main.py
in the pull request upon the next review request.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The issue with redundant files is still present.
# write your code here | ||
sorted_variables = { | ||
"mutable": [ | ||
my_favourite_films, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The issue is not resolved, your pull request should not contain files other than app/main.py
. Visit a Q&A session to resolve this issue and request another review after other files are not present.
14b04fa
to
648871f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job!
@@ -5,3 +5,4 @@ | |||
.DS_Store | |||
venv/ | |||
.pytest_cache/ | |||
tests/__pycache__/test_main.cpython-39-pytest-7.1.3.pyc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add a blank line
No description provided.