-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Add --cog-path
CLI argument
#6510
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
base: V3/develop
Are you sure you want to change the base?
Conversation
This PR does not behave as outlined in the issue. |
Oops, good catch! I'll take another look at this. |
This argument takes a list of strings (can be used multiple times for more than one value), validates that the path provided exists, and adds the path via `self._cog_mgr.add_path()`, so that cogs can be loaded from that path. Closes Cog-Creators#6506
So, I propose the following: |
Why not just store it in memory during the run the flag is used? Why does the disk need to be used at all? |
Description of the changes
This PR adds an argument to
redbot
that takes a list of strings (can be used multiple times for more than one value), validates that the path provided exists, and adds the path viaself._cog_mgr.add_path()
, so that cogs can be loaded from that path.Closes #6506
Have the changes in this PR been tested?
Not thoroughly