-
-
Notifications
You must be signed in to change notification settings - Fork 287
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
Shaderlab remove editor properties #1825
Shaderlab remove editor properties #1825
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #1825 +/- ##
=======================================
Coverage 65.04% 65.04%
=======================================
Files 473 472 -1
Lines 23762 23686 -76
Branches 3367 3364 -3
=======================================
- Hits 15455 15407 -48
+ Misses 7105 7085 -20
+ Partials 1202 1194 -8
☔ View full report in Codecov by Sentry. |
parseShader(shaderSource: string) { | ||
return AstNodeUtils.parseShader(shaderSource); | ||
const editorPropertiesRegex = /EditorProperties\s+\{[^}]*?\}/; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why need replace to empty? just ignore is not OK?
* main: (29 commits) Fix(ShaderLab): make `UsePass` directive compatible with all builtin shader names (#1855) feat: ParticleGenerator support simulationSpeed (#1854) fix: sprite destroy (#1852) "v1.1.0-beta.18" Fix quaternion to euler bug (#1851) Fix ColorOverLifetimeModule mode invalid bug (#1850) fix: no autoplay when set animatorController after enable (#1843) Fix material destoroy bug (#1847) Fix Sprite can't batch bug caused by material instance (#1830) "v1.1.0-beta.17" fix: glTF animation parse bug (#1845) fix: mesh decoder (#1844) "v1.1.0-beta.16" Fix blendShape crossFade bug (#1841) fix: ignore ambient in 2d template (#1836) "v1.1.0-beta.15" Fix blendShapeAnimation crossFade bug (#1840) fix: ignore case of asset url (#1809) `Shaderlab` remove editor properties (#1825) fix: ktx2 texture support (#1829) ... # Conflicts: # packages/core/src/particle/ParticleRenderer.ts
Please check if the PR fulfills these requirements
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
rm the
EditorProperties
module parsing logic in ShaderLab #1803