You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@kracekumar I have added the required changes locally. However, Flake8 keeps giving me errors for star-imports in files like .settings/dev.py. So should I correct them or ignore them before making my PR?
Activity
sivaa commentedon Jun 23, 2017
@kracekumar Are you referring to minimum chars?
kracekumar commentedon Jun 23, 2017
@sivaa Oh, yes. But 15 characters is low but can be configurable.
prakharchoudhary commentedon Jun 25, 2017
@kracekumar I am new to open source and would like to contribute to this project. Can I work on this issue?
ChillarAnand commentedon Jun 25, 2017
Please go ahead @prakharchoudhary
kracekumar commentedon Jun 26, 2017
@prakharchoudhary We'd love to have new contributors. Like @ChillarAnand said, please go ahead and start the work.
Feel free to discuss, doubts, and your plan.
prakharchoudhary commentedon Jun 29, 2017
@kracekumar I have added the required changes locally. However, Flake8 keeps giving me errors for star-imports in files like .settings/dev.py. So should I correct them or ignore them before making my PR?
RishabhJain2018 commentedon Jun 30, 2017
@prakharchoudhary Use
# noqa: ignore=F405as a comment in star imports line to ignore flake8.onlinejudge95 commentedon Apr 11, 2019
Hey guys, what is the status of this issue. Willing to take it up if required.
zerothabhishek commentedon Apr 12, 2019
Hey @onlinejudge95, the PR #537 has been already submitted for this. We can test it ourselves, and merge if it looks good