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

DirectX 12 triangle (SDL2 window) #69

Merged

Conversation

karl-zylinski
Copy link
Collaborator

@karl-zylinski karl-zylinski commented Jan 11, 2025

  • Have you added the example to the CI at .github/workflows/check.yml?

Side note:
I know the d3d11 example is in sdl folder. I'll move that one to the directx folder in a separate PR after this one has been merged. It makes more sense that d3d examples live in the directx folder. The SDL stuff is just for opening the window; the meat of those examples is d3d.

@karl-zylinski karl-zylinski force-pushed the directx12-sdl-triangle branch from e530c2f to ea059e2 Compare January 11, 2025 18:30
@karl-zylinski
Copy link
Collaborator Author

Build error looks like CI error. The line it reports does not contain that code.

@lsr351
Copy link

lsr351 commented Jan 11, 2025

I saw the discord message and look into it. CI uses latest master commit on Odin right? Seems like the latest commit to that file introduced spaces.
https://github.com/odin-lang/examples/blob/master/.github/workflows/check.yml#L26
https://github.com/odin-lang/Odin/blob/c30cdaa7b089df1af788b97fc90833463771293c/vendor/directx/d3d12/d3d12.odin#L1583

@karl-zylinski
Copy link
Collaborator Author

Thanks, I had forgotten to sync my Odin fork.

@karl-zylinski karl-zylinski merged commit efe801b into odin-lang:master Jan 14, 2025
1 check passed
@karl-zylinski karl-zylinski deleted the directx12-sdl-triangle branch January 14, 2025 10:01
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.

2 participants