Skip to content

BF: Slashes in project names were causing Pavlovia sync to error#7017

Merged
TEParsons merged 2 commits intopsychopy:devfrom
TEParsons:dev-bf-pavlovia-slash
Dec 19, 2024
Merged

BF: Slashes in project names were causing Pavlovia sync to error#7017
TEParsons merged 2 commits intopsychopy:devfrom
TEParsons:dev-bf-pavlovia-slash

Conversation

@TEParsons
Copy link
Copy Markdown
Contributor

If the project was originally synced via Builder then this never happens - as Builder strips any extraneous / from the name on creation. However, if the project was cloned via url in command line and the url had a / at the end, the project then couldn't be opened in Builder.

@codecov
Copy link
Copy Markdown

codecov bot commented Dec 18, 2024

Codecov Report

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

Project coverage is 41.64%. Comparing base (7fe212a) to head (9261654).
Report is 31 commits behind head on dev.

❗ There is a different number of reports uploaded between BASE (7fe212a) and HEAD (9261654). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (7fe212a) HEAD (9261654)
4 3
Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #7017      +/-   ##
==========================================
- Coverage   50.96%   41.64%   -9.32%     
==========================================
  Files         340      340              
  Lines       62332    62335       +3     
==========================================
- Hits        31769    25961    -5808     
- Misses      30563    36374    +5811     
Components Coverage Δ
app ∅ <ø> (∅)
boilerplate ∅ <ø> (∅)
library ∅ <ø> (∅)
vm-safe library ∅ <ø> (∅)

@TEParsons TEParsons merged commit b2d7f5f into psychopy:dev Dec 19, 2024
@TEParsons TEParsons deleted the dev-bf-pavlovia-slash 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