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

feat(readme): add section for report troubleshooting #119

Merged
merged 4 commits into from
Feb 15, 2022

Conversation

dvoegeli
Copy link
Contributor

@dvoegeli dvoegeli commented Dec 9, 2021

resolves #114

What Changed

  • add section for report troubleshooting
    • completion time
    • stuck in queue
    • manually accepting proposed mappings
  • fix markdown lint issues

@sonarcloud
Copy link

sonarcloud bot commented Dec 9, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@codecov
Copy link

codecov bot commented Dec 9, 2021

Codecov Report

Merging #119 (a174333) into master (1c9ff5d) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #119   +/-   ##
=======================================
  Coverage   95.58%   95.58%           
=======================================
  Files           8        8           
  Lines         748      748           
=======================================
  Hits          715      715           
  Misses         33       33           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1c9ff5d...a174333. Read the comment docs.

@mehaase mehaase requested a review from emmanvg February 11, 2022 18:16
@mehaase mehaase self-assigned this Feb 11, 2022
@mehaase
Copy link
Contributor

mehaase commented Feb 11, 2022

I merged in the master branch so that this PR is mergeable. There were some conflicts with @emmanvg's edits to the README, so I kept the updated developer setup instructions from @emmanvg and then incorporate the rest of the changes from @dvoegeli.

@sonarcloud
Copy link

sonarcloud bot commented Feb 15, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Copy link
Contributor

@emmanvg emmanvg left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@mehaase mehaase merged commit 54bc5a5 into master Feb 15, 2022
@mehaase mehaase deleted the readme/114_report_completion_time branch February 15, 2022 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upload Report completion
3 participants