-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
feat: add solgun bot trades spellbook #5805
feat: add solgun bot trades spellbook #5805
Conversation
Workflow run id 8774841360 approved. |
Workflow run id 8774841437 approved. |
Tests passed now. |
Workflow run id 8788532192 approved. |
Workflow run id 8788532160 approved. |
great! is there anything left for me to do to get this merged? |
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.
looks good ser, thank you
i'll plan to do some merges tomorrow |
Added spellbook for solgun bot trades using the new filestructure by @Hosuke
giving me some weird error though, hope someone can tell me what i need to fix 😆