Skip to content

BF: Downgrade "git not installed" error to a warning#7086

Merged
TEParsons merged 1 commit intopsychopy:devfrom
TEParsons:dev-bf-git-non-fatal
Feb 6, 2025
Merged

BF: Downgrade "git not installed" error to a warning#7086
TEParsons merged 1 commit intopsychopy:devfrom
TEParsons:dev-bf-git-non-fatal

Conversation

@TEParsons
Copy link
Copy Markdown
Contributor

Because we need git just to check whether there's a linked Pavlovia project upon e.g. opening a file, erroring out whenever we hit a point where we need git and it isn't installed makes the app unusable without git. In theory git should always be there, but recently some versions of OSX have made the installer unable to install git along with PsychoPy.

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 6, 2025

Codecov Report

Attention: Patch coverage is 14.28571% with 6 lines in your changes missing coverage. Please review.

Project coverage is 49.99%. Comparing base (49bc0ce) to head (e0e5b98).
Report is 7 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #7086      +/-   ##
==========================================
- Coverage   49.99%   49.99%   -0.01%     
==========================================
  Files         347      347              
  Lines       64062    64066       +4     
==========================================
+ Hits        32028    32029       +1     
- Misses      32034    32037       +3     
Components Coverage Δ
app ∅ <ø> (∅)
boilerplate ∅ <ø> (∅)
library ∅ <ø> (∅)
vm-safe library ∅ <ø> (∅)

@TEParsons TEParsons merged commit 7445dd3 into psychopy:dev Feb 6, 2025
7 checks passed
@TEParsons TEParsons deleted the dev-bf-git-non-fatal branch February 12, 2025 16:57
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.

1 participant