Skip to content

fix initialization for CADENCE_WEB_PORT in dev#1230

Draft
Assem-Uber wants to merge 1 commit intocadence-workflow:masterfrom
Assem-Uber:fix/development-web-port
Draft

fix initialization for CADENCE_WEB_PORT in dev#1230
Assem-Uber wants to merge 1 commit intocadence-workflow:masterfrom
Assem-Uber:fix/development-web-port

Conversation

@Assem-Uber
Copy link
Copy Markdown
Contributor

@Assem-Uber Assem-Uber commented Apr 1, 2026

Summary

Fix initialization of CADENCE_WEB_PORT env variable to unify the value of the port in the dynamic code value & the value that dev server starts with. The fix is not needed for production docker images as the environment variable is initialized correctly.

Details

The web server used to start on port 8088 while accessing the value in dynamic config was 3000. Setting the environment variable before running the server makes sure that both starting port and dynamic config value is 8088

@gitar-bot
Copy link
Copy Markdown

gitar-bot bot commented Apr 1, 2026

CI failed: PR title validation failure: 'fix initialization for CADENCE_WEB_PORT in dev' does not follow the required conventional commit format with release type prefix.

Overview

One configuration failure identified across 1 analyzed log. The PR title validation action failed because the title lacks the required release type prefix for semantic versioning. This is a blocking issue that must be resolved before the PR can proceed.

Failures

PR Title Does Not Follow Conventional Commit Format (confidence: high)

  • Type: configuration
  • Affected jobs: 69563393479
  • Related to change: yes
  • Root cause: The PR title 'fix initialization for CADENCE_WEB_PORT in dev' includes the conventional commit type 'fix' but lacks the required scope prefix or release type indicator expected by the amannn/action-semantic-pull-request validation action. The semantic release validator requires a more specific format to determine the release type (major, minor, patch).
  • Suggested fix: Reword the PR title to include a scope in parentheses. Examples: 'fix(web): initialization for CADENCE_WEB_PORT in dev' or 'fix(dev): CADENCE_WEB_PORT initialization'. Refer to https://www.conventionalcommits.org/ for correct formatting. Alternatively, if this is not a releasable change, use 'chore: fix initialization for CADENCE_WEB_PORT in dev'.

Summary

  • Change-related failures: 1 — PR title validation failure preventing merge
  • Infrastructure/flaky failures: 0
  • Recommended action: Update the PR title to follow conventional commit format with a scope prefix, then the validation check will pass and CI can proceed.
Code Review ✅ Approved

Fixes initialization for CADENCE_WEB_PORT in the development environment. No issues found.

Tip

Comment Gitar fix CI or enable auto-apply: gitar auto-apply:on

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

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.

1 participant