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

Change precedence in rules to make location after proper casing. #81304

Conversation

ajreckof
Copy link
Member

@ajreckof ajreckof commented Sep 4, 2023

Not sure if it is wanted or not might need a proposal. See #81104 (comment) on why this might be a good idea.

Just posting the PR as I was the one making this part of the code and knew how to change it if need be

@AThousandShips
Copy link
Member

This breaks unit tests, needs to be checked

@ajreckof
Copy link
Member Author

ajreckof commented Sep 4, 2023

oh yeah forgot I had made some unit test on those that would folllow the rules so need to update those. I'll do that tonight.

@ajreckof ajreckof force-pushed the Change-precedence-in-rules-to-make-location-after-proper-casing- branch from c087bf9 to 95317d2 Compare September 5, 2023 12:56
@ajreckof ajreckof requested a review from a team as a code owner September 5, 2023 12:56
@YuriSizov YuriSizov requested a review from a team September 5, 2023 13:44
@YuriSizov YuriSizov modified the milestones: 4.x, 4.2 Sep 5, 2023
@ajreckof ajreckof force-pushed the Change-precedence-in-rules-to-make-location-after-proper-casing- branch from 95317d2 to f8121bd Compare September 5, 2023 13:51
tests/scene/test_code_edit.h Outdated Show resolved Hide resolved
tests/scene/test_code_edit.h Outdated Show resolved Hide resolved
@ajreckof ajreckof force-pushed the Change-precedence-in-rules-to-make-location-after-proper-casing- branch from f8121bd to 364aac7 Compare September 5, 2023 14:03
@ajreckof ajreckof force-pushed the Change-precedence-in-rules-to-make-location-after-proper-casing- branch from 364aac7 to 19632ea Compare September 5, 2023 14:16
Copy link
Member

@akien-mga akien-mga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense to me.

@akien-mga akien-mga merged commit d013fc3 into godotengine:master Sep 7, 2023
15 checks passed
@akien-mga
Copy link
Member

Thanks!

@ajreckof ajreckof deleted the Change-precedence-in-rules-to-make-location-after-proper-casing- branch December 23, 2023 00:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants