-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding scripts & documentation for the SC calls executor #412
Conversation
iulianpascalau
commented
Jan 17, 2025
•
edited
Loading
edited
- added scripts for the SC calls executor
- added documentation for the SC calls executor
- fixed the flag for providing sc proxy addresses
- added CLI.md file & the generating script
…into scripts-docs-sc-calls-executor
- fixed the flag for providing sc proxy addresses - added CLI.md file & the generating script
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Scripts & docs reviewed.
Makefile
Outdated
echo "Error - please update all CLI.md files by running the 'cli-docs' or 'check-cli-md' from Makefile!"; \ | ||
exit 1; \ | ||
fi |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing new line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added
Makefile
Outdated
cd ./cmd && bash ./CLI.md.sh |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can be reused cli-docs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reused
cmd/scCallsExecutor/CLI.md
Outdated
|
||
GLOBAL OPTIONS: | ||
--working-directory directory This flag specifies the directory where the node will store databases and logs. | ||
--log-level level(s) This flag specifies the logger level(s). It can contain multiple comma-separated value. For example, if set to *:INFO the logs for all packages will have the INFO level. However, if set to *:INFO,api:DEBUG the logs for all packages will have the INFO level, excepting the api package which will receive a DEBUG log level. (default: "*:DEBUG") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
--log-level level(s) This flag specifies the logger level(s). It can contain multiple comma-separated value. For example, if set to *:INFO the logs for all packages will have the INFO level. However, if set to *:INFO,api:DEBUG the logs for all packages will have the INFO level, excepting the api package which will receive a DEBUG log level. (default: "*:DEBUG") | |
--log-level level(s) This flag specifies the logger level(s). It can contain multiple comma-separated values. For example, if set to *:INFO the logs for all packages will have the INFO level. However, if set to *:INFO,api:DEBUG the logs for all packages will have the INFO level, excepting the api package which will receive a DEBUG log level. (default: "*:DEBUG") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changed
cmd/scCallsExecutor/CLI.md
Outdated
--working-directory directory This flag specifies the directory where the node will store databases and logs. | ||
--log-level level(s) This flag specifies the logger level(s). It can contain multiple comma-separated value. For example, if set to *:INFO the logs for all packages will have the INFO level. However, if set to *:INFO,api:DEBUG the logs for all packages will have the INFO level, excepting the api package which will receive a DEBUG log level. (default: "*:DEBUG") | ||
--disable-ansi-color Boolean option for disabling ANSI colors in the logging system. | ||
--config [path] The [path] for the main configuration file. This TOML file contain the main configurations such as monitored SC, gateway URL, timings and so on (default: "config/config.toml") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
--config [path] The [path] for the main configuration file. This TOML file contain the main configurations such as monitored SC, gateway URL, timings and so on (default: "config/config.toml") | |
--config [path] The [path] for the main configuration file. This TOML file contains the main configurations such as monitored SC, gateway URL, timings and so on (default: "config/config.toml") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changed
cmd/scCallsExecutor/README.md
Outdated
throttling might occur without the access token. | ||
|
||
The `EXTRA_FLAGS` can contain extra flags to be called whenever the application is started. | ||
The complete list of the cli command can be found [here](./cmd/scCallsExecutor/CLI.md) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The complete list of the cli command can be found [here](./cmd/scCallsExecutor/CLI.md) | |
The complete list of the cli commands can be found [here](./cmd/scCallsExecutor/CLI.md). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changed
HUMAN_TIME=$(date -d @"$GITHUB_API_RESET") | ||
|
||
# Allow user to override the current version to be used | ||
OVERRIDE_VER="" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing new line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added