Skip to content
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: list bots by network #187

Merged
merged 5 commits into from
Jan 14, 2025
Merged

feat: list bots by network #187

merged 5 commits into from
Jan 14, 2025

Conversation

dtdang
Copy link
Contributor

@dtdang dtdang commented Jan 10, 2025

What I did

Change bots list command so that bots are listed by <ecosystem> and <network-provider>.

$ sbc bots list
apeworx:
    ethereum-mainnet:
      - timemachine-bot
      - vars-bot
    ethereum-sepolia:
      - sepolia-bot

fixes: SBK-622

How I did it

How to verify it

Checklist

  • Passes all linting checks (pre-commit and CI jobs)
  • New test cases have been added and are passing
  • Documentation has been updated
  • PR title follows Conventional Commit standard (will be automatically included in the changelog)

@dtdang dtdang requested a review from fubuloubu January 10, 2025 23:32
johnson2427
johnson2427 previously approved these changes Jan 11, 2025
fubuloubu
fubuloubu previously approved these changes Jan 11, 2025
silverback/_cli.py Outdated Show resolved Hide resolved
@dtdang dtdang dismissed stale reviews from fubuloubu and johnson2427 via ee9600a January 13, 2025 17:58
@dtdang dtdang enabled auto-merge (squash) January 13, 2025 18:09
@dtdang dtdang disabled auto-merge January 13, 2025 20:02
@dtdang
Copy link
Contributor Author

dtdang commented Jan 14, 2025

$ sbc bots list
apeworx:
    ethereum-mainnet:
      - sepolia-bot
      - timemachine-bot
      - vars-bot
    ethereum-sepolia:
      - sepolia-bot

List command will now show bots of the same name on multiple networks.

@dtdang dtdang enabled auto-merge (squash) January 14, 2025 18:12
silverback/cluster/client.py Show resolved Hide resolved
silverback/_cli.py Show resolved Hide resolved
@dtdang dtdang merged commit f67ea6c into ApeWorX:main Jan 14, 2025
13 checks passed
@dtdang dtdang deleted the fix/bots-list branch January 14, 2025 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants