Skip to content

airframe-di: Support in-class bind[X] for classes with DISupport trait#816

Merged
xerial merged 3 commits intowvlet:masterfrom
xerial:in-class-bind
Nov 21, 2019
Merged

airframe-di: Support in-class bind[X] for classes with DISupport trait#816
xerial merged 3 commits intowvlet:masterfrom
xerial:in-class-bind

Conversation

@xerial
Copy link
Copy Markdown
Member

@xerial xerial commented Nov 20, 2019

No description provided.

@xerial xerial mentioned this pull request Nov 20, 2019
66 tasks
@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 20, 2019

Codecov Report

Merging #816 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #816      +/-   ##
==========================================
+ Coverage   81.03%   81.03%   +<.01%     
==========================================
  Files         224      224              
  Lines        8893     8894       +1     
  Branches      616      616              
==========================================
+ Hits         7206     7207       +1     
  Misses       1687     1687
Impacted Files Coverage Δ
.../main/scala/wvlet/airframe/AirframeException.scala 85.71% <100%> (ø) ⬆️
...rframe/src/main/scala/wvlet/airframe/Session.scala 92% <100%> (ø) ⬆️
.../main/scala/wvlet/airframe/launcher/Launcher.scala 92% <0%> (ø) ⬆️
...a/wvlet/airframe/launcher/HelpMessagePrinter.scala 96.55% <0%> (+0.06%) ⬆️

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 39ad0e1...e82f725. Read the comment docs.

Copy link
Copy Markdown
Collaborator

@shimamoto shimamoto left a comment

Choose a reason for hiding this comment

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

Left one minor comment, but basically it's good!

Comment thread docs/src/main/tut/docs/internals.md Outdated
@xerial xerial merged commit c3ce56b into wvlet:master Nov 21, 2019
@xerial xerial deleted the in-class-bind branch November 21, 2019 07:14
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.

2 participants