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

DOC: Update documentation of settings and downloads modules #80

Merged
merged 9 commits into from
Feb 22, 2023
Merged

DOC: Update documentation of settings and downloads modules #80

merged 9 commits into from
Feb 22, 2023

Conversation

lboucin
Copy link
Collaborator

@lboucin lboucin commented Feb 14, 2023

No description provided.

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Feb 14, 2023
@github-actions github-actions bot added the testing Anything related to tests label Feb 22, 2023
@codecov-commenter
Copy link

codecov-commenter commented Feb 22, 2023 •
edited
Loading

Codecov Report

Merging #80 (3fcf568) into main (ae2bac2) will decrease coverage by 0.06%.
The diff coverage is 48.00%.

@@            Coverage Diff             @@
##             main      #80      +/-   ##
==========================================
- Coverage   87.06%   87.01%   -0.06%     
==========================================
  Files           9        9              
  Lines        1794     1794              
==========================================
- Hits         1562     1561       -1     
- Misses        232      233       +1

PipKat
PipKat approved these changes Feb 22, 2023
View reviewed changes
Copy link
Member

@PipKat PipKat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a few comments!

@lboucin lboucin merged commit 0aa6d17 into main Feb 22, 2023
@lboucin lboucin deleted the doc/overall_edit_settings branch February 22, 2023 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@PipKat PipKat PipKat approved these changes

@chrpetre chrpetre Awaiting requested review from chrpetre

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation testing Anything related to tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@lboucin @codecov-commenter @PipKat