search

MRChemSoft / mrchem

MultiResolution Chemistry
GNU Lesser General Public License v3.0
27 stars 21 forks source link

Change fetch-depth #348

Closed robertodr closed 3 years ago

robertodr commented 3 years ago

See here: https://github.com/danger/danger/issues/1103#issuecomment-635539947

Minazo is partially healed:

  1. If there are no comments to post, the job completes successfully.
  2. If there are comments to post, the job fails because the bot's permissions are not sufficient to comment on open PRs. I have changed the event that triggers Minazo from pull_request to pull_request_target, which presumably exposes secrets correctly. However, changing triggers is not picked up until the PR is merged...
codecov[bot] commented 3 years ago

Codecov Report

Merging #348 (071940b) into master (1d086cf) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #348   +/-   ##
=======================================
  Coverage   67.66%   67.66%           
=======================================
  Files         160      160           
  Lines       12334    12334           
=======================================
  Hits         8346     8346           
  Misses       3988     3988

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 1d086cf...071940b. Read the comment docs.