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

Update flux_msg_cmp(3) and flux_recv(3) given struct flux_match changes #946

Merged
merged 3 commits into from Jan 11, 2017

Conversation

Projects
None yet
4 participants
@chu11
Copy link
Contributor

chu11 commented Jan 11, 2017

No description provided.

chu11 added some commits Jan 11, 2017

doc/flux_msg_cmp(3): Fix invalid documentation
Function name was listed incorrectly.
doc/flux_msg_cmp(3): Update matchtag usage
Update for matchtag usage per commit

dcec73a

Fixes #935
doc/flux_recv(3): Update flux_match
Update struct flux_match per changes in:

dcec73a

Add pointer to flux_msg_cmp(3) for more details of match
parameter usage.

@garlick garlick added the review label Jan 11, 2017

@codecov-io

This comment has been minimized.

Copy link

codecov-io commented Jan 11, 2017

Current coverage is 76.04% (diff: 100%)

Merging #946 into master will decrease coverage by 0.01%

@@             master       #946   diff @@
==========================================
  Files           149        149          
  Lines         25962      25962          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits          19747      19742     -5   
- Misses         6215       6220     +5   
  Partials          0          0          

Powered by Codecov. Last update ae1c252...512369f

@coveralls

This comment has been minimized.

Copy link

coveralls commented Jan 11, 2017

Coverage Status

Coverage decreased (-0.02%) to 76.327% when pulling 512369f on chu11:issue935 into ae1c252 on flux-framework:master.

@garlick

This comment has been minimized.

Copy link
Member

garlick commented Jan 11, 2017

Excellent, thanks!

@garlick garlick merged commit c55822f into flux-framework:master Jan 11, 2017

4 checks passed

codecov/patch Coverage not affected when comparing ae1c252...512369f
Details
codecov/project Absolute coverage decreased by -0.01% but relative coverage increased by +23.93% compared to ae1c252
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage decreased (-0.02%) to 76.327%
Details

@garlick garlick removed the review label Jan 11, 2017

@grondo grondo referenced this pull request Mar 28, 2017

Closed

0.7.0 Release Notes #1019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.