Skip to content

MAINT: use netlify-cli for sending html to netlify - #599

Closed
mmcky wants to merge 3 commits into
mainfrom
maint-netlify
Closed

mmcky wants to merge 3 commits into
mainfrom
maint-netlify

Conversation

@mmcky

@mmcky mmcky commented Sep 9, 2025

Copy link
Copy Markdown
Contributor

This PR uses netlify-cli to see if we can improve our netlify html previews.

@netlify

netlify Bot commented Sep 9, 2025

Copy link
Copy Markdown

Deploy Preview for sunny-cactus-210e3e ready!

Name Link
🔨 Latest commit 06a2993
🔍 Latest deploy log https://app.netlify.com/projects/sunny-cactus-210e3e/deploys/68bf9287fcbdcb00087d7b0f
😎 Deploy Preview https://deploy-preview-599--sunny-cactus-210e3e.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@mmcky
mmcky requested a review from Copilot September 9, 2025 02:34

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 PR modernizes the Netlify deployment process by replacing the nwtgck/actions-netlify GitHub Action with the official Netlify CLI. The change aims to improve reliability and control over HTML preview deployments.

Key changes:

  • Replaces third-party Netlify action with official CLI tooling
  • Adds custom PR commenting and deployment status management
  • Introduces Node.js setup to support Netlify CLI installation

Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
@github-actions

github-actions Bot commented Sep 9, 2025

Copy link
Copy Markdown

@github-actions
github-actions Bot temporarily deployed to pull request September 9, 2025 02:35 Inactive
mmcky and others added 2 commits September 9, 2025 12:35
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@mmcky

mmcky commented Sep 9, 2025

Copy link
Copy Markdown
Contributor Author

@copilot can you review the failing ci.yml and find a fix?

@github-actions
github-actions Bot temporarily deployed to pull request September 9, 2025 02:49 Inactive
@mmcky mmcky added the testing label Sep 9, 2025
@mmcky

mmcky commented Sep 10, 2025

Copy link
Copy Markdown
Contributor Author

Closing and working on changes in #604

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants