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

CI: fix using Kaggle datasets #160

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

CI: fix using Kaggle datasets #160

wants to merge 6 commits into from

Conversation

Borda
Copy link
Member

@Borda Borda commented Apr 15, 2022

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 401 - Unauthorized
Needed for #125

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 bug / fix Something isn't working ci/cd labels Apr 15, 2022
@codecov
Copy link

codecov bot commented Apr 15, 2022

Codecov Report

Merging #160 (d80627c) into main (caebc8f) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@         Coverage Diff         @@
##           main   #160   +/-   ##
===================================
  Coverage    73%    73%           
===================================
  Files         2      2           
  Lines       382    382           
===================================
  Hits        280    280           
  Misses      102    102           

@stale stale bot added the won't fix This will not be worked on label Jun 23, 2022
@Lightning-AI Lightning-AI deleted a comment from stale bot Jun 29, 2022
@stale stale bot removed the won't fix This will not be worked on label Jun 29, 2022
@stale stale bot closed this Jul 22, 2022
@Borda Borda reopened this Jul 23, 2022
@Borda Borda self-assigned this Aug 15, 2022
@Borda Borda force-pushed the main branch 2 times, most recently from d4acf6f to ac8f7ba Compare July 19, 2024 21:12
@Borda
Copy link
Member Author

Borda commented Jul 22, 2024

actual issue: Kaggle/kaggle-api#521
revert 3f96b5f

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug / fix Something isn't working ci/cd
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant