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

Project setting to control node name casing #7509

Merged
merged 1 commit into from
Jan 12, 2017
Merged

Project setting to control node name casing #7509

merged 1 commit into from
Jan 12, 2017

Conversation

RayKoopa
Copy link
Contributor

As discussed in #7501, I modified my changes to make this a project setting.

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.

Perfect :)

@akien-mga akien-mga added this to the 3.0 milestone Jan 12, 2017
@akien-mga akien-mga merged commit 45e3e23 into godotengine:2.2 Jan 12, 2017
@akien-mga
Copy link
Member

Just noticed that you had made this PR against the 2.2 branch - this one is not meant for development, it's just a legacy branch for people that need its features without the current brokenness of the master branch, but new developments should be done first in master, and then cherry-picked to other branches if relevant. I'll cherry-pick this one for master, but please keep this in mind for your next PR ;)

@RayKoopa
Copy link
Contributor Author

I will, I was kinda confused myself. I could not get the master branch to run well on my Windows system, always crashed right after the project list, so I had to stick with this... =(

@RayKoopa RayKoopa deleted the new_node_name_casing branch January 12, 2017 19:45
@akien-mga
Copy link
Member

True, it's not optimal for development right now, it should settle in a couple of weeks.

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.

2 participants