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

Add keepers-minimal project #17

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add keepers-minimal project #17

wants to merge 1 commit into from

Conversation

andrejrakic
Copy link
Contributor

This project aims to show minimal examples of using Chainlink Keepers to automate some of the contracts from the defi-minimal repository.

Please do check the README file carefully. Thanks

@andrejrakic andrejrakic self-assigned this Jul 22, 2022
Copy link

@PatrickAlphaC PatrickAlphaC left a comment

Choose a reason for hiding this comment

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

Could we add some tests? It'll be a lot easier to see what these are trying to do with tests.

And maybe we rename the folder to "defi-automation" I feel like "keepers-minimal" doesn't really do it justice, what do you think?

3. Run tests

```shell
yarn test

Choose a reason for hiding this comment

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

Looks like there are no tests right now?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure thing, will add tests soon

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