Skip to content
This repository has been archived by the owner on Mar 11, 2023. It is now read-only.

Conversation

rtyley
Copy link
Contributor

@rtyley rtyley commented Mar 20, 2019

Following on from #237 (comment) this is the smallest change required to get the constructor-overriding hack in this gist to work:

https://gist.github.com/rtyley/dc341992a6104f4b499834fd2d10dfad

…hack

Following on from DanielaSfregola#237 (comment)
this is the smallest change required to get the constructor-overriding hack
in this gist to work:

https://gist.github.com/rtyley/dc341992a6104f4b499834fd2d10dfad
@codecov
Copy link

codecov bot commented Mar 20, 2019

Codecov Report

Merging #239 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #239   +/-   ##
=======================================
  Coverage   94.28%   94.28%           
=======================================
  Files          76       76           
  Lines        1173     1173           
  Branches       12       12           
=======================================
  Hits         1106     1106           
  Misses         67       67
Impacted Files Coverage Δ
.../danielasfregola/twitter4s/TwitterRestClient.scala 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 6a5ab13...2bb0ee3. Read the comment docs.

@DanielaSfregola DanielaSfregola added this to the 6.0.1 milestone Mar 20, 2019
Copy link
Owner

@DanielaSfregola DanielaSfregola left a comment

Choose a reason for hiding this comment

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

Hi @rtyley,
thanks for your contribution! Will try to do a patch release as soon as possible (version 6.0.1?).

LGTM

@DanielaSfregola DanielaSfregola merged commit 21e1eb7 into DanielaSfregola:master Mar 20, 2019
@DanielaSfregola
Copy link
Owner

DanielaSfregola commented Mar 20, 2019

I have now released twitter4s 6.0.1 with this patch -- it should be in maven central in a few minutes

@rtyley
Copy link
Contributor Author

rtyley commented Mar 20, 2019

Great! Thanks so much!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants