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

Remove temporary project conversion #99479

Merged
merged 1 commit into from
Feb 18, 2025

Conversation

Sauermann
Copy link
Contributor

@Sauermann Sauermann commented Nov 20, 2024

This project conversion affects only projects, that were edited after #97707 (v4.4-dev4) and provides an easy way to upgrade these projects.

It seems reasonable to assume, that at that time of the next stable Godot-release all affected projects have been converted.
So I would like to suggest to merge this PR and remove the project conversion shortly before the next stable release,

For projects, that haven't been converted, we can provide an upgrade-strategy, that that consists of opening the project once with v4.4-dev5.

Follow-up to/Implements #99449 (review)

@Sauermann Sauermann added this to the 4.4 milestone Nov 20, 2024
@Sauermann Sauermann requested a review from a team as a code owner November 20, 2024 19:29
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.

I queued this for 4.4-rc1 for now.

This project conversion should be removed before the next stable release,
because it affects only projects opened during dev-builds.
@Sauermann Sauermann force-pushed the fix-remove-project-conversion branch from 6c0c122 to ee2930c Compare December 13, 2024 23:39
@Sauermann
Copy link
Contributor Author

I have updated the PR to additionally revert #100334.

@Repiteo Repiteo merged commit 24dccd4 into godotengine:master Feb 18, 2025
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Feb 18, 2025

Thanks!

@Sauermann Sauermann deleted the fix-remove-project-conversion branch February 19, 2025 22:14
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.

3 participants