Skip to content

Conversation

@FilipRudy
Copy link
Contributor

Description

Changes proposed in this pull request:

  • As in title

Related issue(s)

Resolves #2554

@FilipRudy FilipRudy requested a review from a team as a code owner November 24, 2025 20:20
@FilipRudy FilipRudy marked this pull request as draft November 24, 2025 20:20
@github-actions
Copy link

github-actions bot commented Nov 24, 2025

⚠️ Proposed changes verification failed

This message means some of your changes may interrupt repository standards.

Follow these requirements:

  • ❌ Documentation is out of date:
     M docs/user/gen-docs/_sidebar.md
     M docs/user/gen-docs/_sidebar.ts
     M docs/user/gen-docs/kyma.md
     M go.mod
     M go.sum
    ?? docs/user/gen-docs/kyma_dashboard.md
    ?? docs/user/gen-docs/kyma_dashboard_start.md
    Please run make docs and commit the changes.
  • ❌ Found usage of os.Stdout, os.Stderr or fmt.Print in code:
    ./internal/dashboard/dashboard.go:			fmt.Printf("Docker installation seems to be Docker Desktop. Appending '%s' to env variables of container
    ", dockerEnv)
    ./internal/dashboard/dashboard.go:	fmt.Print("
    ")
    ./internal/dashboard/dashboard.go:			fmt.Printf("Operating system seems to be Linux. Changing the Docker network mode to 'host'")
    ./internal/cmd/docker/docker.go:	streamer := streams.NewOut(os.Stdout)
    ./internal/cmd/docker/docker.go:		dstout, dsterr = os.Stdout, os.Stderr
    Please use the internal/out package for output handling instead.

Find more detailed information in the verify / standards (pull_request_target) action.

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.

Add ability to run Kyma Dashboard locally

1 participant