Skip to content

Conversation

@tstanmay13
Copy link
Contributor

Description

Linear ticket: Closes FER-7220: [Java, intuit] update multi env names

Fixes environment name normalization to maintain original casing (e.g., "E2E", "QAL", "Production") for multi-URL environments, addressing customer linting requirements.

Changes:

  • Modified EnvironmentGenerator.java (line 154) to use .getOriginalName() instead of .getScreamingSnakeCase().getSafeName()

Backward Compatibility:
Not a breaking change—single-URL environments still use SCREAMING_SNAKE_CASE. Only affects new adopters of multi-URL environments (PR #9219). This is only Intuit so we don't really need to make this a breaking change for them.

Example: "E2E" now generates as E2E instead of E_2_E.

@tstanmay13 tstanmay13 marked this pull request as ready for review October 17, 2025 01:38
@aditya-arolkar-swe aditya-arolkar-swe self-requested a review October 20, 2025 12:11
@chrismallinson chrismallinson force-pushed the tanmay/intuit-env-name-change branch from 128c414 to be5ceb8 Compare October 20, 2025 12:54
@tstanmay13 tstanmay13 enabled auto-merge (squash) October 20, 2025 16:30
@tstanmay13 tstanmay13 disabled auto-merge October 20, 2025 16:31
@tstanmay13 tstanmay13 merged commit 9ab6487 into main Oct 20, 2025
218 of 231 checks passed
@tstanmay13 tstanmay13 deleted the tanmay/intuit-env-name-change branch October 20, 2025 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants