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

Added /ok-to-test function, reformatted CI #849

Merged
merged 1 commit into from Jun 10, 2022

Conversation

Wxl19980214
Copy link
Contributor

@Wxl19980214 Wxl19980214 commented Jun 9, 2022

Describe your changes

Added three files, ci-integration.yaml, ci-unit.yaml, ok-to-test.yaml.
Still need to test it on some branch, ideally master branch.

Issue ticket number and link

Closes #717

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have added thorough tests (when necessary).
  • I have added the right documentation (when needed). Product update? If yes, write one line about this update.

added cron

cron temp test

remove cron test

removed testing jobs

fixed typo

cron for unit

removed cover all
@Wxl19980214 Wxl19980214 marked this pull request as ready for review June 9, 2022 19:14
@Wxl19980214 Wxl19980214 mentioned this pull request Jun 10, 2022
3 tasks
@idomic idomic merged commit 64b4dec into ploomber:master Jun 10, 2022
Wxl19980214 added a commit to Wxl19980214/ploomber that referenced this pull request Jun 10, 2022
added cron

cron temp test

remove cron test

removed testing jobs

fixed typo

cron for unit

removed cover all

merge into master (ploomber#849)

added cron

cron temp test

remove cron test

removed testing jobs

fixed typo

cron for unit

removed cover all

Update ok-to-test.yml

removed old cis
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.

PR CI failing
2 participants