Skip to content
This repository has been archived by the owner on May 24, 2021. It is now read-only.

feat: full sync for contributor email #28

Merged
merged 5 commits into from
Jan 19, 2021
Merged

feat: full sync for contributor email #28

merged 5 commits into from
Jan 19, 2021

Conversation

Mini256
Copy link
Member

@Mini256 Mini256 commented Jan 18, 2021

CheckList

  • feat: full sync for contributor email
  • feat: increamental sync for contributor email
  • feat: avoid repeated processing contributor email when pr merged
  • docs: add doc

@todo
Copy link

todo bot commented Jan 18, 2021

Ensure that all pull requests from contributors can be obtained.

// TODO: Ensure that all pull requests from contributors can be obtained.
return await this.pullRepository.find({
user: login,
});


This comment was generated by todo based on a TODO comment in 9a7a351 in #28. cc @ti-community-infra.

@codecov
Copy link

codecov bot commented Jan 18, 2021

Codecov Report

Merging #28 (f2a46a5) into main (616c1b9) will increase coverage by 89.03%.
The diff coverage is 89.33%.

Impacted file tree graph

@@            Coverage Diff            @@
##           main      #28       +/-   ##
=========================================
+ Coverage      0   89.03%   +89.03%     
=========================================
  Files         0       13       +13     
  Lines         0      301      +301     
  Branches      0       35       +35     
=========================================
+ Hits          0      268      +268     
- Misses        0       23       +23     
- Partials      0       10       +10     
Impacted Files Coverage Δ
src/repositories/ContributorRepository.ts 36.84% <36.84%> (ø)
src/utils/parser.ts 71.42% <83.33%> (ø)
src/services/IssueService.ts 88.37% <88.37%> (ø)
src/services/ContributorService.ts 91.89% <91.89%> (ø)
src/services/CommentService.ts 92.53% <92.53%> (ø)
src/services/PullService.ts 93.33% <93.33%> (ø)
src/db/entities/Issue.ts 94.44% <94.44%> (ø)
src/db/entities/Pull.ts 94.73% <94.73%> (ø)
src/db/entities/Comment.ts 95.23% <95.23%> (ø)
src/common/token.ts 100.00% <100.00%> (ø)
... and 16 more

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 c332258...f2a46a5. Read the comment docs.

@Mini256 Mini256 changed the title WIP: feat: full sync for contributor email feat: full sync for contributor email Jan 19, 2021
@Mini256
Copy link
Member Author

Mini256 commented Jan 19, 2021

hold for fix a bug

Copy link
Contributor

@hi-rustin hi-rustin left a comment

Choose a reason for hiding this comment

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

rest LGTM.

@todo
Copy link

todo bot commented Jan 19, 2021

Avoid repeated processing contributor email when pr merged.

// TODO: Avoid repeated processing contributor email when pr merged.
// Synchronize contributor mailboxes when pull request is merged.
if (pullRequest.merged_at !== null) {
// Fetch the patch of pull request.
const patch = await getPullRequestPatch(
context.pullRequest(),


This comment was generated by todo based on a TODO comment in f2a46a5 in #28. cc @ti-community-infra.

@Mini256
Copy link
Member Author

Mini256 commented Jan 30, 2021

🎉 This PR is included in version 1.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

None yet

2 participants