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.

Sign up for GitHub

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

Jump to bottom

Add attachment ID to body. #9

Merged
merged 5 commits into from
Sep 26, 2016
Merged

Add attachment ID to body. #9

merged 5 commits into from
Sep 26, 2016

Conversation

johansja
Copy link
Contributor

@johansja johansja commented Sep 21, 2016 •
edited
Loading

Plus some minor cleanup by fixing compiler warning and linter recommendations.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.006%) to 98.208% when pulling 8e5d3dd on JohanSJA:dev into 7063dd3 on craigp:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-2.1%) to 96.057% when pulling 1aa66d7 on JohanSJA:dev into 7063dd3 on craigp:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.006%) to 98.208% when pulling 6f17eac on JohanSJA:dev into 7063dd3 on craigp:master.

@johansja johansja changed the title Fix compiler warnings. Add attachment ID to body. Sep 22, 2016
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.006%) to 98.195% when pulling f778e94 on JohanSJA:dev into 7063dd3 on craigp:master.

@johansja johansja mentioned this pull request Sep 22, 2016
Closed
@craigp
Copy link
Owner

craigp commented Sep 26, 2016

Thanks @johansja, this looks great 👍

@craigp craigp merged commit 02037d5 into craigp:master Sep 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@johansja @coveralls @craigp