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

Exclude OpenXR from macOS in Godot < 4.2; run codegen-full in full-ci #405

Merged
merged 4 commits into from
Sep 10, 2023

Conversation

Bromeon
Copy link
Member

@Bromeon Bromeon commented Sep 10, 2023

Godot does not support OpenXR* classes under macOS for versions <= 4.1.x, we thus conditionally exclude them given that configuration.

Now also runs CI on the full generated code (in full-ci during merging, not in minimal-ci). This would catch the problem with macOS.

@Bromeon Bromeon added bug c: engine Godot classes (nodes, resources, ...) labels Sep 10, 2023
@GodotRust
Copy link

API docs are being generated and will be shortly available at: https://godot-rust.github.io/docs/gdext/pr-405

@Bromeon Bromeon added this pull request to the merge queue Sep 10, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Sep 10, 2023
@Bromeon Bromeon added this pull request to the merge queue Sep 10, 2023
@Bromeon
Copy link
Member Author

Bromeon commented Sep 10, 2023

Already working on getting compile-times down (especially for clippy in CI, this even caused a timeout)...

Merged via the queue into master with commit b51b005 Sep 10, 2023
14 checks passed
@Bromeon Bromeon deleted the bugfix/mac-openxr branch September 10, 2023 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug c: engine Godot classes (nodes, resources, ...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants