Skip to content
This repository was archived by the owner on Aug 28, 2025. It is now read-only.

Conversation

@Borda
Copy link
Contributor

@Borda Borda commented Jun 14, 2021

Before submitting

  • Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
  • Did you make sure to update the docs?
  • Did you write any new necessary tests?

What does this PR do?

Fixes #32

PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

@Borda Borda added enhancement New feature or request ci/cd labels Jun 14, 2021
@Borda Borda enabled auto-merge (squash) June 14, 2021 21:05
@Borda Borda requested a review from ethanwharris June 14, 2021 21:14
@Borda Borda mentioned this pull request Jun 14, 2021
3 tasks
@Borda Borda requested a review from edgarriba June 14, 2021 21:30
Copy link
Member

@ethanwharris ethanwharris left a comment

Choose a reason for hiding this comment

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

LGTM :)

@Borda Borda disabled auto-merge June 14, 2021 22:20
@Borda Borda merged commit fbb2261 into main Jun 14, 2021
@Borda Borda deleted the cd/ipynb-req branch June 14, 2021 22:20
pl-ghost pushed a commit that referenced this pull request Jun 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

ci/cd enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

define requirements for notebooks

2 participants