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/virtual protocol #202

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

kahwaipd
Copy link

Added virtualprotocol/ignis tool

@kahwaipd
Copy link
Author

@0xArdi kindly review


def run(**kwargs) -> Tuple[Optional[str], Optional[Dict[str, Any]], Any, Any]:
"""Run the task"""
api_keys = kwargs["api_keys"]['ignis'].split(":")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kahwaipd where can we get such an api key for testing purposes?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll share it in Telegram group.

@0xArdi
Copy link
Collaborator

0xArdi commented Mar 29, 2024

@kahwaipd the tool looks good, can you address the failing checks please?
I see that poetry.lock is not updated. Also, make sure you lock the packages before you push, you can use the following command:

autonomy packages lock

@0xArdi
Copy link
Collaborator

0xArdi commented Apr 15, 2024

@kahwaipd can you please address the check failures? If you have any questions let me know.

@kahwaipd
Copy link
Author

@0xArdi I have executed the "autonomy packages lock" , can you please check?

@0xArdi
Copy link
Collaborator

0xArdi commented Jul 17, 2024

@kahwaipd any chance you can resolve some of these issues?

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.

2 participants