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.

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

Lack of pep8 python files #114

Open
2 of 6 tasks
SparshKhanna0001 opened this issue Nov 9, 2024 · 0 comments
Open
2 of 6 tasks

Lack of pep8 python files #114

SparshKhanna0001 opened this issue Nov 9, 2024 · 0 comments

Comments

@SparshKhanna0001
Copy link

SparshKhanna0001 commented Nov 9, 2024

Issue Title

The python files Video.py Edit.py and Face.py isn't pep8 format this causes issues with the pre-commit pr ci bot.

Type

  • Big
  • Feature Request
  • Info
  • Bug
  • Documentation
  • [ ]
  • Other (please specify):

Description

When I did Submitted the poll request for the issue 104 adding statistics to the read me pre commit report caused an error or reported the PR incompatible The reason being Face.py, Video.py and Edit.py not being pep8 compatible. Clearly the bot isn't function the way it shud. The solution lies in altering the bot preferences to ignore some kind of files that don't require pep 8 compatibility And also form Defaulted files.

Expected Behavior

I expected the cold request to Green signal to my pr but the pre commit bot caused issue, which wasn't even related to the pull request

Current Behavior

If the files aren't formatted correctly and the bot preferences aren't set this is gonna cause a lot of trouble to other pull requests

Steps to Reproduce

  1. Just try to commit a md file or any other file too

Environment

Not required

Possible Solution (if you have any in mind)

As described above, add gssoc and hactoerbfest label and i will sort it out.

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

No branches or pull requests

1 participant