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

fix: plugin logging #139

Merged
merged 1 commit into from
May 29, 2024
Merged

fix: plugin logging #139

merged 1 commit into from
May 29, 2024

Conversation

Spazzy757
Copy link
Contributor

With go-plugin it is hard to debug the plugins, therefore this change should allow for more robust logging. We should in the future make this more confiurable and aligined with the internal logging system

@Spazzy757 Spazzy757 requested a review from gabibeyer May 29, 2024 08:00
@Spazzy757 Spazzy757 added the plugins issues related to the plugin system label May 29, 2024
Copy link

netlify bot commented May 29, 2024

Deploy Preview for aether-docs canceled.

Name Link
🔨 Latest commit 5498b7a
🔍 Latest deploy log https://app.netlify.com/sites/aether-docs/deploys/6656e594c7c84b00082fa981

@Spazzy757 Spazzy757 force-pushed the fix-add-better-plugin-logging branch from 5aca204 to b6f3049 Compare May 29, 2024 08:15
@Spazzy757 Spazzy757 force-pushed the fix-add-better-plugin-logging branch from b6f3049 to 780cede Compare May 29, 2024 08:19
With go-plugin it is hard to debug the plugins, therefore this change
should allow for more robust logging. We should in the future make this
more confiurable and aligined with the internal logging system

Signed-off-by: Spazzy <brendankamp757@gmail.com>
@Spazzy757 Spazzy757 force-pushed the fix-add-better-plugin-logging branch from 9a40aac to 5498b7a Compare May 29, 2024 08:21
@Spazzy757 Spazzy757 merged commit eb498b7 into main May 29, 2024
9 checks passed
@Spazzy757 Spazzy757 deleted the fix-add-better-plugin-logging branch May 29, 2024 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plugins issues related to the plugin system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants