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

Pass issue bodies and PR base/head branches into templating method #164

Merged

Conversation

kyeah
Copy link
Contributor

@kyeah kyeah commented May 10, 2018

Closes #131, closes #161.

Since we're using listPullRequests for PRs now, we can pull the PR base/head directly from issue.

As noted in #161, it would be nice to pass the full object along, at least if the config provides a custom templating function, but I'm keeping the scope small for this.

@codecov-io
Copy link

codecov-io commented May 10, 2018

Codecov Report

Merging #164 into master will decrease coverage by 45.79%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #164      +/-   ##
==========================================
- Coverage   84.05%   38.26%   -45.8%     
==========================================
  Files           7        7              
  Lines         345      345              
==========================================
- Hits          290      132     -158     
- Misses         55      213     +158
Impacted Files Coverage Δ
lib/src/Gren.js 5.28% <ø> (-75.49%) ⬇️
lib/src/GitHubInfo.js 80.95% <0%> (-4.77%) ⬇️

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 025ad77...1a769c2. Read the comment docs.

@alexcanessa alexcanessa merged commit 837d4b2 into github-tools:master May 23, 2018
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.

Pass issue descriptions into templates Template: PR branch name?
3 participants