Pylint Score Increased for apps/hosts

[Akshat453]

Fixes Issue : #4557

Pylint Score After Changes: Your code has been rated at 10.00/10

[RishabhJain2018]

Let's not disbale E0402. Please try and find a fix for it.

[Akshat453]

i have to fine fix for both E0402 and E0401 right?

[RishabhJain2018]

I already fixed E0401 in #4556

[Akshat453]

okay I'll find fix for E0402 and let you know

[RishabhJain2018]

Please resolve the conflicts too!

[Akshat453]

Hello @RishabhJain2018 I have fixed the E0402 import error by changing relative imports (like from challenges.models) to absolute imports (from apps.challenges.models) to improve the Pylint score. However, this change is now causing the build to fail

[RishabhJain2018]

Please resolve the conflicts.

[Akshat453]

@RishabhJain2018 i have resolved the conflicts

[Akshat453]

Hey @RishabhJain2018,

The build is currently failing due to the following error:
RuntimeError: Conflicting 'challenge_participant_teams' models in application 'challenges': <class 'challenges.models.Challenge_participant_teams'> and <class 'apps.challenges.models.Challenge_participant_teams'>.

This conflict arises because in most files the model is imported as "from challenges.models import Challenge", but in utils.py, I had to change it to "from apps.challenges.models import Challenge" to resolve an import error during runtime.

Would it be better to update all instances across the codebase to use "from apps.challenges.models import Challenge" for consistency, or should I instead revert my change and stick with "from challenges.models import Challenge" if that works

[Akshat453]

Hey @RishabhJain2018,

The build is currently failing due to the following error: RuntimeError: Conflicting 'challenge_participant_teams' models in application 'challenges': <class 'challenges.models.Challenge_participant_teams'> and <class 'apps.challenges.models.Challenge_participant_teams'>.

This conflict arises because in most files the model is imported as "from challenges.models import Challenge", but in utils.py, I had to change it to "from apps.challenges.models import Challenge" to resolve an import error during runtime.

Would it be better to update all instances across the codebase to use "from apps.challenges.models import Challenge" for consistency, or should I instead revert my change and stick with "from challenges.models import Challenge" if that works

Hey @RishabhJain2018 ,Can you review this