Skip to content

Use existing plugin.cfg in configuration dialog - #79723

Merged
akien-mga merged 1 commit into
godotengine:masterfrom
KoBeWi:tfw_you_single-linedly_implement_a_proposal😎
Apr 26, 2024

Hidden character warning

The head ref may contain hidden characters: "tfw_you_single-linedly_implement_a_proposal\ud83d\ude0e"
Merged

Use existing plugin.cfg in configuration dialog#79723
akien-mga merged 1 commit into
godotengine:masterfrom
KoBeWi:tfw_you_single-linedly_implement_a_proposal😎

Conversation

@KoBeWi

@KoBeWi KoBeWi commented Jul 20, 2023

Copy link
Copy Markdown
Member

@IceflowRE

Copy link
Copy Markdown
Contributor

As already stated in the issue, wouldn't it be better to avoid loading the file twice?

@KoBeWi

KoBeWi commented Jul 20, 2023

Copy link
Copy Markdown
Member Author

It doesn't matter. It's still better than project metadata, which is loaded with every access 🙃

@akien-mga akien-mga left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good. Would rebase for good measure.

@akien-mga akien-mga modified the milestones: 4.x, 4.3 Apr 26, 2024
@KoBeWi
KoBeWi force-pushed the tfw_you_single-linedly_implement_a_proposal branch from 15c4602 to 7b3da23 Compare April 26, 2024 12:38
@akien-mga
akien-mga merged commit 593e308 into godotengine:master Apr 26, 2024
@akien-mga

Copy link
Copy Markdown
Member

Thanks!

@KoBeWi
KoBeWi deleted the tfw_you_single-linedly_implement_a_proposal😎 branch April 26, 2024 13:32
@akien-mga akien-mga changed the title Use existing plugin.cfg Use existing plugin.cfg in configuration dialog May 1, 2024
BendyLand pushed a commit to BendyLand/voltaire that referenced this pull request Aug 2, 2026
…ly_implement_a_proposal😎

Use existing `plugin.cfg`
BendyLand pushed a commit to BendyLand/voltaire that referenced this pull request Aug 2, 2026
…ly_implement_a_proposal😎

Use existing `plugin.cfg`
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.

Persistent additional data in plugin configuration files

3 participants