Removal of the Ersilia Current command

[Ajoke23]

Thank you for taking your time to contribute to Ersilia, just a few checks before we proceed

• Have you followed the guidelines in our Contribution Guide
• Have you written new tests for your core changes, as applicable?
• Have you successfully ran tests with your changes locally?

Description

Remove the Ersilia current command

Changes to be made

• Deleting of the current command line i.e cmd.current()
• Removal the cmd function which is the command implememtation code in
  Status

To do
None

Related to #1267

[Ajoke23]

@DhanshreeA I will be expecting your feedback

[P-Mefut]

hi @Ajoke23, looks good but we still have a def for current in the cmd.py files

def current(self): m = importlib.import_module("ersilia.cli.commands.current") m.current_cmd()

that's line 32 - 34 which should be removed if your taking out the command

[Ajoke23]

Hi @DhanshreeA
The following steps were taking to successfully remove current command:

• removed cmd.current() from create_cli.py
• deleted the curent.py file since the code there aren't needed
• I removed the def function for current found in cmd.py file
  I tested the code locally and the result shows that current command has been successfully removed

[Ajoke23]

hi @Ajoke23, looks good but we still have a def for current in the cmd.py files

def current(self): m = importlib.import_module("ersilia.cli.commands.current") m.current_cmd()

that's line 32 - 34 which should be removed if your taking out the command

Thanks for the suggestion.

[dzumii]

Hello @Ajoke23, I can see you already created another pull request here, can you confirm we can disregard and close this then?

[Ajoke23]

Hello @Ajoke23, I can see you already created another pull request here, can you confirm we can disregard and close this then?

Yes, you can close this particular pull request #1282