Skip to content

Fix #276: Add make open and improve make up UX#276

Merged
samdark merged 2 commits intomasterfrom
make-ux
Apr 21, 2026
Merged

Fix #276: Add make open and improve make up UX#276
samdark merged 2 commits intomasterfrom
make-ux

Conversation

@samdark
Copy link
Copy Markdown
Member

@samdark samdark commented Apr 21, 2026

Q A
Is bugfix?
New feature? ✔️
Breaks BC?
Fixed issues -

It adds make open that opens the app in the browser.

Also, it enhances the output of make up:

image

And in case of failure:

image

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 21, 2026 16:17
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 21, 2026

Important

Review skipped

Auto reviews are disabled on this repository. 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: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: cd77b541-99ad-463d-880f-4455a3e47397

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
  • Commit unit tests in branch make-ux

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.

Comment thread CHANGELOG.md Outdated
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 21, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.91%. Comparing base (2fc01ab) to head (78945d0).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #276   +/-   ##
=========================================
  Coverage     84.91%   84.91%           
  Complexity       52       52           
=========================================
  Files            16       16           
  Lines           179      179           
=========================================
  Hits            152      152           
  Misses           27       27           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a convenience make open target and improves make up output/diagnostics for the Docker-based dev environment, aligning the documented local URL with the HTTP-only Caddy setup.

Changes:

  • Enhance make up to wait for containers, print the resolved localhost URL, and provide more diagnostics on failure.
  • Add make open to open the running app URL in the system default browser.
  • Update README/Changelog to document the new target and the correct http://localhost URL.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
Makefile Implements enhanced up behavior and introduces the new open target.
README.md Documents make open and updates the Docker access URL to HTTP.
CHANGELOG.md Adds an entry describing the UX improvements and new command.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Makefile
Comment thread Makefile
Comment thread Makefile
Comment thread Makefile
@samdark samdark changed the title Enhance make up and add make open Fix #276: Add make open and improve make up UX Apr 21, 2026
@samdark samdark merged commit 130d988 into master Apr 21, 2026
19 checks passed
@samdark samdark deleted the make-ux branch April 21, 2026 22:17
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.

2 participants