Conversation
TRomesh
left a comment
TRomesh
left a comment
There was a problem hiding this comment.
Do not put import statements here like this they should be done on the top. Refactor the code.
lib/compute.js
Outdated
| this._compute = ncProviders.azure.compute(options); | ||
| } else { | ||
| throw new Error("Please specify a provider by flag -p --provider"); | ||
| const emoji = require("node-emoji"); |
There was a problem hiding this comment.
Do not put import statements here like this they should be done on the top.
There was a problem hiding this comment.
Since its a common thing you can move it to helper.js and import it as a function
lib/database.js
Outdated
| this._database = nodecloud.azure.compute(options); | ||
| } else { | ||
| throw new Error("Please specify a provider by flag -p --provider"); | ||
| const emoji = require("node-emoji"); |
There was a problem hiding this comment.
Do not put import statements here like this they should be done on the top.
There was a problem hiding this comment.
Since its a common thing you can move it to helper.js and import it as a function
lib/network.js
Outdated
| @@ -1,26 +1,27 @@ | |||
| class Network { | |||
| constructor(program, nodecloud, options) { | |||
| const emoji = require("node-emoji"); | |||
There was a problem hiding this comment.
Do not put import statements here like this they should be done on the top.
README.md
Outdated
|
|
||
| nodecloud-cli is an unified command line interface for open cloud based on [nodecloud](https://github.com/cloudlibz/nodecloud). nodecloud-cli supports cloud providers like AWS, Azure, GCP and many more. | ||
|
|
||
| Very Simple to Use. |
There was a problem hiding this comment.
Can you revert the change to README.md
4 participants
I have added emojis to the cli