acquia / cli

Acquia CLI
GNU General Public License v2.0
42 stars 47 forks source link

CLI-1104: Undefined array key url #1560

Closed danepowell closed 1 year ago

danepowell commented 1 year ago

I'd prefer to strictly validate the git config file, but the syntax / schema isn't rigidly defined by Git itself. We could also use git commands to parse it, but that's probably overkill.

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (482dc20) 91.74% compared to head (7b3d556) 91.74%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1560 +/- ## ========================================= Coverage 91.74% 91.74% - Complexity 1655 1656 +1 ========================================= Files 110 110 Lines 5938 5939 +1 ========================================= + Hits 5448 5449 +1 Misses 490 490 ``` | [Impacted Files](https://app.codecov.io/gh/acquia/cli/pull/1560?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=acquia) | Coverage Δ | | |---|---|---| | [src/Command/CommandBase.php](https://app.codecov.io/gh/acquia/cli/pull/1560?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=acquia#diff-c3JjL0NvbW1hbmQvQ29tbWFuZEJhc2UucGhw) | `94.13% <100.00%> (+<0.01%)` | :arrow_up: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.