search

sailuh / kaiaulu

An R package for mining software repositories
http://itm0.shidler.hawaii.edu/kaiaulu
Mozilla Public License 2.0
20 stars 13 forks source link

Scitools Understand Parser #309

Open RavenMarQ opened 2 months ago

RavenMarQ commented 2 months ago

This is to test Committing and PR

carlosparadis commented 1 month ago

@RavenMarQ just a reminder the notebook should have more than just code, please check with @daomcgill or ask her for a code review if needed.

carlosparadis commented 1 month ago

In addition to the above, see CONTRIBUTING.md on how to edit NEWS.md, and DESCRIPTION. A lot of files that shouldn't be added have been pushed in this PR and those that should be edited were not.

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 0% with 54 lines in your changes missing coverage. Please review.

Project coverage is 37.94%. Comparing base (2bc8d14) to head (857d3c4). Report is 4 commits behind head on master.

Files with missing lines Patch % Lines
R/src.R 0.00% 54 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #309 +/- ## ========================================== - Coverage 39.79% 37.94% -1.85% ========================================== Files 20 20 Lines 3091 3355 +264 ========================================== + Hits 1230 1273 +43 - Misses 1861 2082 +221 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

carlosparadis commented 1 week ago

@daomcgill the GitHub Actions problem was incidentally isolated: The moment I merged The GitHub Refresher, master started passing. I thought GitHub Actions broke something, but the problem is deeper and in our end. Notice how @RavenMarQ PR also passed just now after I merged the passing master. @beydlern @crepesAlot please merge master to your PRs that are still open. master is currently passing, as so is this PR. If yours is not passing, I can't merge.

Ping me on your PRs that do not pass master so we can diagnose. If you had to merge Dao's PR, then the issue on her code likely spread to yours.