Skip to content

Conversation

@9renpoto
Copy link
Contributor

@9renpoto 9renpoto commented Apr 2, 2018

TL;DR

Connect to #438

@ghost ghost assigned 9renpoto Apr 2, 2018
@ghost ghost added the in progress label Apr 2, 2018
@9renpoto 9renpoto added this to the v1.0.0 milestone Apr 2, 2018
@9renpoto 9renpoto requested review from dkimura and shoota April 2, 2018 06:34
* feat(examples) Added writing-plugin examples

* refactor(packages) use @userdive/provider
Copy link

@dkimura dkimura left a comment

Choose a reason for hiding this comment

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

細かいことだけど、https://github.com/userdive/agent.js/blob/feature/provide/README.mdpackages@userdive/provider を追加しなくても良い?

@codecov
Copy link

codecov bot commented Apr 2, 2018

Codecov Report

Merging #439 into master will decrease coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #439      +/-   ##
==========================================
- Coverage   91.63%   91.62%   -0.02%     
==========================================
  Files          19       19              
  Lines         550      549       -1     
  Branches       97       97              
==========================================
- Hits          504      503       -1     
  Misses         25       25              
  Partials       21       21
Impacted Files Coverage Δ
packages/provider/src/index.ts 100% <100%> (ø)

Copy link

@dkimura dkimura left a comment

Choose a reason for hiding this comment

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

全体的には良さそう。

@9renpoto 9renpoto added QA and removed in progress labels Apr 2, 2018
@dkimura
Copy link

dkimura commented Apr 2, 2018

細かいことだけど、/provide/README.md@feature の packages に @userdive/provider を追加しなくても良い?

↑ は ↓ とのこと。

404 link, publishする前に追加できないんですよね

@dkimura dkimura merged commit 264e6cf into master Apr 2, 2018
@ghost ghost removed the QA label Apr 2, 2018
@dkimura dkimura deleted the feature/provide branch April 2, 2018 07:53
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.

3 participants