Skip to content

_add_builtin_plugins after setting from env#5566

Merged
adhami3310 merged 1 commit into
mainfrom
_add_builtin_plugins-after-setting-from-env
Jul 12, 2025
Merged

_add_builtin_plugins after setting from env#5566
adhami3310 merged 1 commit into
mainfrom
_add_builtin_plugins-after-setting-from-env

Conversation

@adhami3310
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

PR Summary

Reordered plugin initialization in reflex/config.py to respect environment variables before loading built-in plugins.

  • Modified plugin initialization sequence in reflex/config.py to ensure environment variables are processed before calling _add_builtin_plugins(), allowing for proper plugin configuration based on environment settings

1 file reviewed, no comments
Edit PR Review Bot Settings | Greptile

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Jul 11, 2025

CodSpeed Performance Report

Merging #5566 will not alter performance

Comparing _add_builtin_plugins-after-setting-from-env (d94c1da) with main (5890982)

Summary

✅ 8 untouched benchmarks

@adhami3310 adhami3310 merged commit 69d6eb5 into main Jul 12, 2025
38 of 41 checks passed
@adhami3310 adhami3310 deleted the _add_builtin_plugins-after-setting-from-env branch July 12, 2025 03:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants