Skip to content

Change port to default 4insight port - #2

Merged
anna-follestad-4ss merged 2 commits into
mainfrom
change-port
Aug 18, 2026
Merged

Change port to default 4insight port#2
anna-follestad-4ss merged 2 commits into
mainfrom
change-port

Conversation

@anna-follestad-4ss

Copy link
Copy Markdown
Collaborator

What changed

The default port was set to 5050, 4insight's default port

Why

The previous version used dash's default port.

Definition of done

CI covers the first two. The rest are yours — tick them because you did them,
not because they were already ticked. See CONTRIBUTING.md for why each is here.

  • pytest passes
  • black is clean
  • PORT is introduced as an env variable in .env
  • PORT is loaded and passed to app.run
  • App is served at localhost:5050 as default.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This pull request updates the Dash app startup configuration to use 4insight’s default port (5050) by default, while allowing the port to be configured via an environment variable.

Changes:

  • Introduces a PORT environment variable (defaulting to 5050) and passes it into app.run(...).
  • Documents the new PORT setting in .env.example.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/app.py Reads PORT from environment (default 5050) and passes it to app.run.
.env.example Adds PORT=5050 to the example environment configuration.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/app.py Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@anna-follestad-4ss
anna-follestad-4ss merged commit 4e2ad9e into main Aug 18, 2026
1 check passed
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.

2 participants