Skip to content

feat: debug tools#266

Merged
alienvspredator merged 1 commit intomainfrom
feat/debug-transport
Mar 27, 2026
Merged

feat: debug tools#266
alienvspredator merged 1 commit intomainfrom
feat/debug-transport

Conversation

@alienvspredator
Copy link
Copy Markdown
Contributor

@alienvspredator alienvspredator commented Mar 27, 2026 •
edited
Loading

This PR adds debug tools and features that can be enabled via an environment variable
"DEBUGFEATURES=feature_name=value,another_feature=some_value".

Inspired by how GODEBUG works:
https://go.dev/doc/godebug

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 27, 2026 •
edited
Loading

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 933a17fe-662f-4c2c-ae49-ac3c41af159e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@alienvspredator alienvspredator requested a review from cb80 March 27, 2026 15:29
cb80
cb80 previously approved these changes Mar 27, 2026
View reviewed changes
@alienvspredator alienvspredator requested a review from cb80 March 27, 2026 15:59
@alienvspredator alienvspredator enabled auto-merge (squash) March 27, 2026 16:17
protectorT1
protectorT1 approved these changes Mar 27, 2026
View reviewed changes
internal/debugtools/settings.go
parse(env)
}

// parse parses DEBUGFEATURES environment variable in the form of k1=v1,k2=v2,k3=v3
Copy link
Copy Markdown
Contributor

@protectorT1 protectorT1 Mar 27, 2026

Choose a reason for hiding this comment

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

minor: typo, double parses

Copy link
Copy Markdown
Contributor Author

@alienvspredator alienvspredator Mar 27, 2026

Choose a reason for hiding this comment

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

It wasn't a typo. It was the format 'function_name (aka parse)' does (parses) something.

@alienvspredator alienvspredator merged commit 3b30b93 into main Mar 27, 2026
7 of 15 checks passed
@alienvspredator alienvspredator deleted the feat/debug-transport branch March 27, 2026 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cb80 cb80 cb80 approved these changes

@protectorT1 protectorT1 protectorT1 approved these changes

Assignees

No one assigned

Labels

feature tests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@alienvspredator @cb80 @protectorT1