Skip to content

fix: remove prefix for variables in flows - #12060

Merged
jordanrfrazier merged 2 commits into
feat/deploymentsfrom
remove-prefix-env-vars
Mar 6, 2026
Merged

fix: remove prefix for variables in flows#12060
jordanrfrazier merged 2 commits into
feat/deploymentsfrom
remove-prefix-env-vars

Conversation

@HimavarshaVS

Copy link
Copy Markdown
Collaborator

No description provided.

@coderabbitai

coderabbitai Bot commented Mar 5, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

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: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 0446f480-aa4b-4c21-96b4-782848011e26

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
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch remove-prefix-env-vars

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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.

@codecov

codecov Bot commented Mar 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (feat/deployments@4fbf6a0). Learn more about missing BASE report.

Additional details and impacted files

Impacted file tree graph

@@                 Coverage Diff                 @@
##             feat/deployments   #12060   +/-   ##
===================================================
  Coverage                    ?   36.26%           
===================================================
  Files                       ?     1581           
  Lines                       ?    77112           
  Branches                    ?    11614           
===================================================
  Hits                        ?    27963           
  Misses                      ?    47577           
  Partials                    ?     1572           
Flag Coverage Δ
lfx 40.71% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@HzaRashid

Copy link
Copy Markdown
Collaborator

lgtm! will wait until your wxo work gets merged. i am about to create a new pr with only the wxo service implementation, will let you know when, so that we can change the base branch of this pr to it

@jordanrfrazier
jordanrfrazier merged commit a2e65f3 into feat/deployments Mar 6, 2026
73 of 100 checks passed
@jordanrfrazier
jordanrfrazier deleted the remove-prefix-env-vars branch March 6, 2026 16:19
@jordanrfrazier
jordanrfrazier restored the remove-prefix-env-vars branch March 6, 2026 16:19
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.

3 participants