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

Added debugLine() method #332

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

aesereht
Copy link

Script pointing the top stack to get script name, function and line.

Script pointing the top stack to get script name, function and line
@Qubus0 Qubus0 changed the base branch from main to development August 24, 2023 15:14
@KANAjetzt KANAjetzt added the enhancement New feature or request label Aug 25, 2023
@KANAjetzt KANAjetzt added this to the v6.3.0 milestone Aug 25, 2023
@KANAjetzt
Copy link
Collaborator

KANAjetzt commented Aug 27, 2023

Thank you for this PR! 🥇

The logs can be sometimes a bit confusing, but in most cases its very helpful.

Not so helpful:

WARNING ModLoader:Deprecated: DEPRECATED: The mod_main.gd _init argument (modLoader = ModLoader) is deprecated. Remove it from your _init to avoid crashes in the next major version. (since version 6.1.0)
Problem File - Source:res://addons/mod_loader/mod_loader.gd - Function:_init - Line:65

Helpful:

WARNING ModLoader:Deprecated: DEPRECATED: The var "mod_data" was removed, use "ModLoaderMod.get_mod_data_all()" instead (since version 6.0.0)
Problem File - Source:res://mods-unpacked/otDan-BetterModList/extensions/ui/menus/title_screen/title_screen_menus.gd - Function:_ready - Line:7

Would be nice if we could move the origin of the deprecation warning to the top of the debugger stack.
image

So the mod dev only gets the deprecation error message and the origin in the debugger.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants