⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

fix: always add the url to json output#7738

Open
minivan wants to merge 6 commits intomainfrom
minivan/fix-astro-feedback
Open

fix: always add the url to json output#7738
minivan wants to merge 6 commits intomainfrom
minivan/fix-astro-feedback

Conversation

@minivan
Copy link
Contributor

@minivan minivan commented Oct 23, 2025

When we removed the --prod as a requirement from the ntl build command, there's a check we missed. It's useful to see the url for deploys.

@minivan minivan requested a review from a team as a code owner October 23, 2025 15:18
@github-actions
Copy link

github-actions bot commented Oct 23, 2025

📊 Benchmark results

Comparing with 803c6ae

  • Dependency count: 1,052 (no change)
  • Package size: 317 MB ⬇️ 0.00% decrease vs. 803c6ae
  • Number of ts-expect-error directives: 366 (no change)

@minivan minivan enabled auto-merge (squash) October 23, 2025 15:59
Copy link
Member

@serhalp serhalp left a comment

Choose a reason for hiding this comment

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

LGTM

@minivan minivan disabled auto-merge October 23, 2025 16:03
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.

3 participants