Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Do not use # in PR build names #796

Merged
merged 2 commits into from
Dec 11, 2024

Conversation

mikeage
Copy link
Member

@mikeage mikeage commented Dec 10, 2024

Currently, this creates a User Data folder at file://C:\Users\foo\AppData\LocalLow\Icosa Foundation\Open Brush-(PR#1234)\ which breaks GLTF loading. This PR removes the # if set, and stops trying to set it in the first place.

@mikeage mikeage added the infrastructure Build or tooling infrastructure label Dec 10, 2024
@mikeage mikeage merged commit b9cd011 into icosa-foundation:main Dec 11, 2024
27 checks passed
@mikeage mikeage deleted the rename_pr_build branch December 11, 2024 07:09
andybak added a commit to icosa-foundation/open-blocks that referenced this pull request Dec 14, 2024
andybak added a commit to icosa-foundation/open-blocks that referenced this pull request Dec 14, 2024
MaoChangYuYan pushed a commit to MaoChangYuYan/OpenBrush_MYT that referenced this pull request Dec 24, 2024
MaoChangYuYan pushed a commit to MaoChangYuYan/OpenBrush_MYT that referenced this pull request Dec 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure Build or tooling infrastructure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants