Skip to content

Fix tags that cause build errors in the WGSL spec#5046

Merged
kainino0x merged 2 commits intogpuweb:mainfrom
mehmetoguzderin:oguz-20250121-fix
Jan 21, 2025
Merged

Fix tags that cause build errors in the WGSL spec#5046
kainino0x merged 2 commits intogpuweb:mainfrom
mehmetoguzderin:oguz-20250121-fix

Conversation

@mehmetoguzderin
Copy link
Copy Markdown
Member

This PR fixes the closing of a few tags which are causing build to fail, blocking PRs such as #5042 due to WGSL spec.

Thank you!

@mehmetoguzderin mehmetoguzderin added the copyediting Pure editorial stuff (copyediting, *.bs file syntax, etc.) label Jan 20, 2025
@mehmetoguzderin
Copy link
Copy Markdown
Member Author

@kainino0x there seems to be similar issues in the API spec too. This PR only handles the WGSL ones.

Thank you!

@mehmetoguzderin mehmetoguzderin enabled auto-merge (squash) January 20, 2025 14:39
@mehmetoguzderin mehmetoguzderin enabled auto-merge (squash) January 20, 2025 14:39
@mehmetoguzderin
Copy link
Copy Markdown
Member Author

mehmetoguzderin commented Jan 20, 2025

It seems like we can't merge with failing CI anymore (this only takes care of WGSL side). I guess we can either merge this first or the API side. Thank you!

@kainino0x kainino0x disabled auto-merge January 21, 2025 18:53
@kainino0x kainino0x merged commit 3dc5313 into gpuweb:main Jan 21, 2025
@kainino0x
Copy link
Copy Markdown
Contributor

Temporarily adjusted the rule so I could land this.

@kainino0x kainino0x added this to the Milestone 1 milestone Jan 21, 2025
@kainino0x kainino0x added the wgsl WebGPU Shading Language Issues label Jan 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

copyediting Pure editorial stuff (copyediting, *.bs file syntax, etc.) wgsl WebGPU Shading Language Issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants