Skip to content

Add GDLint as a workflow and re-format files to fit Godot's style guide - #437

Merged
BenjaTK merged 26 commits into
gaea-godot:2.0from
BenjaTK:gdlint
Oct 15, 2025
Merged

Add GDLint as a workflow and re-format files to fit Godot's style guide#437
BenjaTK merged 26 commits into
gaea-godot:2.0from
BenjaTK:gdlint

Conversation

@BenjaTK

@BenjaTK BenjaTK commented Sep 24, 2025

Copy link
Copy Markdown
Collaborator

No description provided.

@BenjaTK
BenjaTK marked this pull request as ready for review September 24, 2025 18:57
@BenjaTK
BenjaTK marked this pull request as draft September 24, 2025 18:57
@BenjaTK
BenjaTK marked this pull request as ready for review September 24, 2025 19:00
@BenjaTK
BenjaTK marked this pull request as draft September 24, 2025 19:00
@BenjaTK BenjaTK changed the title Add GDLint as a workflow Add GDLint as a workflow and re-format files to fit Godot's style guide Sep 24, 2025
@BenjaTK

BenjaTK commented Sep 24, 2025

Copy link
Copy Markdown
Collaborator Author

We'll have to wait until it's updated to support abstract functions I think

@Zehir

Zehir commented Oct 13, 2025

Copy link
Copy Markdown
Collaborator

@BenjaTK
BenjaTK marked this pull request as ready for review October 14, 2025 12:00
@BenjaTK
BenjaTK marked this pull request as draft October 14, 2025 12:12
@BenjaTK
BenjaTK marked this pull request as ready for review October 14, 2025 19:40
@BenjaTK
BenjaTK requested review from Zehir and cullumi October 14, 2025 19:41

@Zehir Zehir left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Some of the comments are not really related for the context of this PR but I think they are quick performance or cleanup we could do here too

Comment thread addons/gaea/editor/editor_settings.gd
Comment thread addons/gaea/editor/node_popup.gd Outdated
Comment thread addons/gaea/graph/components/argument_editors/range_argument_editor.gd Outdated
Comment thread addons/gaea/graph/graph_nodes/generic_classes/set_operation.gd
Comment thread addons/gaea/graph/graph_nodes/node_resource.gd Outdated
Comment thread addons/gaea/resources/gaea_value.gd
Comment thread testing/rendering/tilemap_test_scene.tscn
Comment thread .gdlintrc
Comment thread .gdlintrc
@BenjaTK BenjaTK added the organization Movement of code around, refactoring, removal of code. label Oct 14, 2025
@BenjaTK
BenjaTK requested a review from Zehir October 14, 2025 21:31

@Zehir Zehir left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@BenjaTK
BenjaTK merged commit 46c762d into gaea-godot:2.0 Oct 15, 2025
5 checks passed
@BenjaTK
BenjaTK deleted the gdlint branch October 16, 2025 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

organization Movement of code around, refactoring, removal of code. repository

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants