Skip to content

👷 Skip Netlify doc publish on PRs#6622

Merged
dubzzz merged 6 commits into
mainfrom
copilot/update-netlify-deployment-workflow
Feb 19, 2026
Merged

👷 Skip Netlify doc publish on PRs#6622
dubzzz merged 6 commits into
mainfrom
copilot/update-netlify-deployment-workflow

Conversation

Copilot AI commented Feb 19, 2026

Copy link
Copy Markdown
Contributor
  • Add if: github.event_name == 'push' condition to publish_documentation_netlify job
  • Remove pull-requests: write permission and PR preview comment step
  • Simplify DRAFT_BUILD env to only check ref
  • Simplify deploy step to single curl command (no file redirect + cat)
  • Fix broken YAML: use folded block scalar for curl command (colons in inline value), fix check_any_package indentation (3→2 spaces)
Original prompt

only publish doc to netlify on merged commit not on pull request, edit workflow accordingly


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@changeset-bot

changeset-bot Bot commented Feb 19, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 0064a27

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copilot AI changed the title [WIP] Update workflow to publish docs only on merged commit 👷 Skip Netlify doc publish on PRs Feb 19, 2026
Copilot AI requested a review from dubzzz February 19, 2026 23:03
Comment thread .github/workflows/build-status.yml Outdated
Comment on lines 560 to 562
run: |
curl -H "Content-Type: application/zip" -H "Authorization: Bearer ${{secrets.NETLIFY_AUTH_TOKEN}}" --data-binary "@artifacts.zip" https://api.netlify.com/api/v1/sites/${{secrets.NETLIFY_SITE_ID}}/deploys?draft=$DRAFT_BUILD_VALUE > deploy.json
cat deploy.json

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

@copilot no more need to curl pipe to file and then cat the file. Everything can be a one-liner on the same line as run:

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Simplified to a single curl command in 9388289.

Copilot AI requested a review from dubzzz February 19, 2026 23:14
Comment thread .github/workflows/build-status.yml
@dubzzz
dubzzz marked this pull request as ready for review February 19, 2026 23:16
@dubzzz
dubzzz enabled auto-merge (squash) February 19, 2026 23:16
@dubzzz

dubzzz commented Feb 19, 2026

Copy link
Copy Markdown
Owner

@copilot action seems to be broken, could you fix it

auto-merge was automatically disabled February 19, 2026 23:39

Head branch was pushed to by a user without write access

Copilot AI commented Feb 19, 2026

Copy link
Copy Markdown
Contributor Author

@copilot action seems to be broken, could you fix it

Fixed in 0064a27. Two issues from when the GitHub suggestion was applied: (1) the inline run: value contained colons which broke YAML parsing — now uses a >- folded block scalar, and (2) check_any_package: had 3 spaces of indentation instead of 2.

Copilot AI requested a review from dubzzz February 19, 2026 23:40
@dubzzz
dubzzz enabled auto-merge (squash) February 19, 2026 23:44
@pkg-pr-new

pkg-pr-new Bot commented Feb 19, 2026

Copy link
Copy Markdown
@fast-check/ava

npm i https://pkg.pr.new/@fast-check/ava@6622

fast-check

npm i https://pkg.pr.new/fast-check@6622

@fast-check/jest

npm i https://pkg.pr.new/@fast-check/jest@6622

@fast-check/packaged

npm i https://pkg.pr.new/@fast-check/packaged@6622

@fast-check/poisoning

npm i https://pkg.pr.new/@fast-check/poisoning@6622

@fast-check/vitest

npm i https://pkg.pr.new/@fast-check/vitest@6622

@fast-check/worker

npm i https://pkg.pr.new/@fast-check/worker@6622

commit: 0064a27

@dubzzz
dubzzz merged commit 5b99e0f into main Feb 19, 2026
49 of 50 checks passed
@dubzzz
dubzzz deleted the copilot/update-netlify-deployment-workflow branch February 19, 2026 23:51
@codecov

codecov Bot commented Feb 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.72%. Comparing base (a9d2de6) to head (0064a27).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6622   +/-   ##
=======================================
  Coverage   94.72%   94.72%           
=======================================
  Files         209      209           
  Lines        5670     5670           
  Branches     1503     1502    -1     
=======================================
  Hits         5371     5371           
  Misses        286      286           
  Partials       13       13           
Flag Coverage Δ
tests 94.72% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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