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

fairness: update headers of source files #113

Merged
merged 1 commit into from Apr 8, 2021

Conversation

cmoussa1
Copy link
Member

@cmoussa1 cmoussa1 commented Apr 6, 2021

The headers for some of the files in the fairness directory need to be updated to the new minimal header and made consistent with the other files in this repo.

This PR updates the headers of some of the files in the fairness directory.

The headers for some of the files in the fairness
directory need to be updated to the new minimal
header and made consistent with the other files in
this repo.

Update the headers of some of the files in the fairness
directory.
@cmoussa1 cmoussa1 added the documentation Improvements or additions to documentation label Apr 6, 2021
@codecov
Copy link

codecov bot commented Apr 6, 2021

Codecov Report

Merging #113 (f7478a7) into master (c8cec66) will not change coverage.
The diff coverage is n/a.

❗ Current head f7478a7 differs from pull request most recent head 693733a. Consider uploading reports for the commit 693733a to get more accurate results

@@           Coverage Diff           @@
##           master     #113   +/-   ##
=======================================
  Coverage   85.37%   85.37%           
=======================================
  Files          13       13           
  Lines         718      718           
=======================================
  Hits          613      613           
  Misses        105      105           
Impacted Files Coverage Δ
src/fairness/account/account.cpp 60.60% <ø> (ø)
src/fairness/account/account.hpp 100.00% <ø> (ø)
src/fairness/reader/test/data_reader_db_test01.cpp 98.52% <ø> (+0.02%) ⬆️
...fairness/weighted_tree/test/weighted_tree_load.cpp 77.55% <ø> (ø)
...irness/weighted_tree/test/weighted_tree_test01.cpp 100.00% <ø> (ø)
src/fairness/weighted_tree/weighted_tree.cpp 86.79% <ø> (ø)
src/fairness/weighted_tree/weighted_tree.hpp 100.00% <ø> (ø)
src/fairness/weighted_tree/weighted_walk.cpp 95.41% <ø> (ø)
src/fairness/weighted_tree/weighted_walk.hpp 100.00% <ø> (ø)
... and 1 more

@dongahn
Copy link
Member

dongahn commented Apr 6, 2021

LGTM.

@cmoussa1
Copy link
Member Author

cmoussa1 commented Apr 6, 2021

Thanks @dongahn!

@cmoussa1 cmoussa1 marked this pull request as ready for review April 6, 2021 20:09
@cmoussa1
Copy link
Member Author

cmoussa1 commented Apr 6, 2021

I'll go ahead and set MWP.

@mergify mergify bot merged commit b38eb9f into flux-framework:master Apr 8, 2021
@cmoussa1 cmoussa1 deleted the update.headers branch August 16, 2021 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation merge-when-passing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants