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

Solve bundle problem on automated tests #268

Merged
merged 4 commits into from
Feb 16, 2021
Merged

Solve bundle problem on automated tests #268

merged 4 commits into from
Feb 16, 2021

Conversation

mcgege
Copy link
Member

@mcgege mcgege commented Feb 16, 2021

remove version pinning from check-utils (bundle problem)

Signed-off-by: Michael Geiger info@mgeiger.de

Signed-off-by: Michael Geiger <info@mgeiger.de>
@mcgege mcgege added the bug label Feb 16, 2021
Signed-off-by: Michael Geiger <info@mgeiger.de>
Signed-off-by: Michael Geiger <info@mgeiger.de>
@codecov-io
Copy link

codecov-io commented Feb 16, 2021

Codecov Report

❗ No coverage uploaded for pull request base (master@25a250e). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #268   +/-   ##
=========================================
  Coverage          ?   53.48%           
=========================================
  Files             ?        4           
  Lines             ?       43           
  Branches          ?        0           
=========================================
  Hits              ?       23           
  Misses            ?       20           
  Partials          ?        0           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 25a250e...178ca94. Read the comment docs.

Signed-off-by: Michael Geiger <info@mgeiger.de>
@schurzi
Copy link
Contributor

schurzi commented Feb 16, 2021

oh nice, you managed to find the problem. I did not have enough patience for fixing this ;)

@mcgege
Copy link
Member Author

mcgege commented Feb 16, 2021

@schurzi @rndmh3ro Do you know where this check continuous-integration/travis-ci is hooked into this project? I'd like to disable it, otherwise you always have to admin merge the PRs

@schurzi
Copy link
Contributor

schurzi commented Feb 16, 2021

Should be under Settings -> Branches there you manage the CI Tests which have to pass, maybe Travis is still marked there.

@rndmh3ro
Copy link
Member

I removed it.

@rndmh3ro rndmh3ro merged commit 90ca82a into master Feb 16, 2021
@rndmh3ro rndmh3ro deleted the gem-error branch February 16, 2021 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants