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

inject-request-scope: Cross-build for Scala 2.13 #553

Conversation

chrisbenincasa
Copy link
Contributor

Problem

inject-request-scope is not cross-built for Scala 2.13.

Solution

Update inject-request-scope module to cross-build for Scala 2.13 using new SBT
settings.

Result

inject-request-scope is cross-built for Scala 2.13.

@codecov
Copy link

codecov bot commented Sep 10, 2020

Codecov Report

Merging #553 into develop will increase coverage by 0.02%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #553      +/-   ##
===========================================
+ Coverage    90.92%   90.94%   +0.02%     
===========================================
  Files          269      269              
  Lines         4872     4872              
  Branches       304      303       -1     
===========================================
+ Hits          4430     4431       +1     
+ Misses         442      441       -1     
Impacted Files Coverage Δ
.../twitter/finatra/thrift/routing/ThriftWarmup.scala 75.00% <0.00%> (ø)
...pp/src/main/scala/com/twitter/inject/app/App.scala 98.50% <0.00%> (+1.49%) ⬆️

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 074cbaa...11009a7. Read the comment docs.

Problem

inject-request-scope is not cross-built for Scala 2.13.

Solution

Update inject-request-scope module to cross-build for Scala 2.13 using
new SBT
settings.

Result

inject-request-scope is cross-built for Scala 2.13.
@cacoco
Copy link
Contributor

cacoco commented Oct 9, 2020

@chrisbenincasa I have to apologize, I lost attribution when merging this here: 1e8dbd1. Really sorry about that. Let me know if you would like me to attempt to fix this.

@chrisbenincasa
Copy link
Contributor Author

chrisbenincasa commented Oct 11, 2020 via email

@cacoco
Copy link
Contributor

cacoco commented Oct 13, 2020

No worries!

Thanks for the work here! Merged in 1e8dbd1.

@cacoco cacoco closed this Oct 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants