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

modules/wreck: Remove hashkey_t type #1706

Merged
merged 1 commit into from Oct 5, 2018

Conversation

Projects
None yet
3 participants
@chu11
Copy link
Contributor

chu11 commented Oct 5, 2018

Per RFC7, remove typdefs that point to fixed length arrays.

Fixes #1703

modules/wreck: Remove hashkey_t type
Per RFC7, remove typdefs that point to fixed length arrays.

Fixes #1703
@codecov-io

This comment has been minimized.

Copy link

codecov-io commented Oct 5, 2018

Codecov Report

Merging #1706 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1706      +/-   ##
==========================================
- Coverage   79.27%   79.26%   -0.01%     
==========================================
  Files         185      185              
  Lines       35042    35042              
==========================================
- Hits        27779    27777       -2     
- Misses       7263     7265       +2
Impacted Files Coverage Δ
src/modules/wreck/wreck_job.c 94.11% <100%> (ø) ⬆️
src/modules/connector-local/local.c 73.16% <0%> (-1.36%) ⬇️
src/cmd/flux-module.c 85.58% <0%> (ø) ⬆️
src/modules/kvs/kvs.c 66% <0%> (+0.15%) ⬆️
src/modules/kvs-watch/kvs-watch.c 75.96% <0%> (+0.82%) ⬆️
@garlick

This comment has been minimized.

Copy link
Member

garlick commented Oct 5, 2018

Thanks!

@garlick garlick merged commit b11ec09 into flux-framework:master Oct 5, 2018

3 checks passed

codecov/patch 100% of diff hit (target 79.27%)
Details
codecov/project Absolute coverage decreased by -<.01% but relative coverage increased by +20.72% compared to b08e53a
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
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.