Fix PyLint Failures

[varshav0119]

Ignoring PyLint failures is probably not the best practice, but we haven't gotten around to fixing them yet 😂 .
The languages/english.py file especially has many issues to fix.

[LustigePerson]

If you could assign me to this issue, I'll have a look.

Actually it seems like you need a complete refactoring.
Without deeper knowledge on whow and where this is integrated I don't think one should start doing this.

• Star imports
• Unused Imports
• Relative Imports (I would suggest to turn this into real modules)
• Functions da do nothing
• Variable naming

[anexas5]

Please assign this issue to me.

[bauripalash]

@FruitVodka

• offtopic 》
  Why you need a temp_dir ,

  aryabota-services/temp_dir/results.json

  Line 1 in bdbd7e2

  [{"stateChanges": [{"row": 1, "column": 2, "dir": "right", "pen": "down", "coloured": [{"position": {"row": 1, "column": 1}}]}]}]

I dont understand the necessity of pushing a temporary directory to the repository.

If the directory is used for something elese, you should rename it appropriately.

[Linta-Iqbal]

Please assign this issue to me.

[varshav0119]

@FruitVodka

• offtopic 》
  Why you need a temp_dir ,

  aryabota-services/temp_dir/results.json

  Line 1 in bdbd7e2

  [{"stateChanges": [{"row": 1, "column": 2, "dir": "right", "pen": "down", "coloured": [{"position": {"row": 1, "column": 1}}]}]}]

I dont understand the necessity of pushing a temporary directory to the repository.

If the directory is used for something elese, you should rename it appropriately.

Hmm the file within it is a temporary file to write down the results, which are picked up and read here to be sent as response to this API call.
Since there's an "exec" of a python program string happening here, couldn't think of a better way to do this.
Did you mean the results file shouldn't be in a directory named temp_dir, or that we shouldn't have a temporary results file in the first place?

[varshav0119]

If you could assign me to this issue, I'll have a look.

Actually it seems like you need a complete refactoring. Without deeper knowledge on whow and where this is integrated I don't think one should start doing this.

• Star imports
• Unused Imports
• Relative Imports (I would suggest to turn this into real modules)
• Functions da do nothing
• Variable naming

@LustigePerson This is integrated with the web app. If you have any questions regarding what certain functions do, or issue with any variable naming, do get back and I'll be happy to clarify!