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

Add open_inputs to the list of exceptions #80

Merged
merged 1 commit into from
Jan 2, 2024

Conversation

Mickeon
Copy link
Contributor

@Mickeon Mickeon commented Jan 2, 2024

Required for godotengine/godot#86713 to be merged.

All tests pass with the original name. But with "open_inputs" they do not.

@qarmin qarmin merged commit e231ae8 into godotengine:3.x Jan 2, 2024
@Mickeon Mickeon deleted the patch-1 branch January 2, 2024 20:26
@Mickeon
Copy link
Contributor Author

Mickeon commented Jan 2, 2024

Very swift, thank you very much!

@AThousandShips
Copy link
Member

AThousandShips commented Jan 2, 2024

Wait, this was the wrong branch surely, the CI checks against the 4.0 branch

Yes indeed this branch is only for 3.x, it has 3.x syntax for GDScript

@qarmin
Copy link
Collaborator

qarmin commented Jan 2, 2024

I wanted to make a cherrypick, similar to what is done in gitlab, but I couldn't find this option via webui and I guess it is only available via gitlab desktop/cli (https://github.com/orgs/community/discussions/61621, isaacs/github#629)

In that case, however, it is rather better to re-create the PR with the appropriate branch

@Mickeon
Copy link
Contributor Author

Mickeon commented Jan 2, 2024

... Sorry for the inconvenience, I was linked the wrong branch indeed. This one could be reverted although it's not exactly harmful

@AThousandShips
Copy link
Member

No harm done 🙂 just got to have it in the right place

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.

3 participants