[ci] Migrate from nwtgck/actions-netlify@v3 to netlify-cli for better PR preview control #604

Copilot · 2025-09-09T02:48:53Z

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

netlify · 2025-09-09T02:49:04Z

✅ Deploy Preview for sunny-cactus-210e3e ready!

Name	Link
🔨 Latest commit	`ba28d05`
🔍 Latest deploy log	https://app.netlify.com/projects/sunny-cactus-210e3e/deploys/68c1fa979c421c0008d234c4
😎 Deploy Preview	https://deploy-preview-604--sunny-cactus-210e3e.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

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

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>

mmcky · 2025-09-09T03:01:23Z

@copilot the purpose of collab.yml is to test execution of code in the google collab environment. Let's remove the netlify preview mechanism from this workflow as it is not required.

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>

Copilot · 2025-09-09T03:03:59Z

@copilot the purpose of collab.yml is to test execution of code in the google collab environment. Let's remove the netlify preview mechanism from this workflow as it is not required.

Removed the Netlify preview mechanism from the collab.yml workflow as requested. The workflow now focuses solely on testing code execution in the Google Colab environment and ends after the execution testing and error reporting steps. Commit: c599088

github-actions · 2025-09-09T03:08:40Z

🚀 Deployed on https://68bf9a32f99df5a60b1d9220--sunny-cactus-210e3e.netlify.app

mmcky · 2025-09-09T03:15:50Z

@copilot the ci.yml workflow is reporting that npm is not installed

Run npm install -g netlify-cli
/home/runner/_work/_temp/cc09cd62-25c2-45ec-8954-ec28f5d2da3f.sh: line 1: npm: command not found
Error: Process completed with exit code 127.

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>

Copilot · 2025-09-09T03:20:24Z

@copilot the ci.yml workflow is reporting that npm is not installed

...

Fixed the npm installation issue by adding Node.js and npm via conda before installing netlify-cli. The workflow was failing because npm wasn't available in the conda environment. Commit: b5b588e

.github/workflows/ci.yml

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>

github-actions · 2025-09-10T05:10:56Z

📖 Netlify Preview Ready!

Preview URL: https://deploy-preview-604--sunny-cactus-210e3e.netlify.app (b1354f8)

📚 Changed Lecture Pages: mle

github-actions · 2025-09-10T05:11:34Z

📖 Netlify Preview Ready!

Preview URL: https://deploy-preview-604--sunny-cactus-210e3e.netlify.app (b1354f8)

📚 Changed Lecture Pages: mle

mmcky · 2025-09-10T05:11:53Z

📖 Netlify Preview Ready!

Preview URL: https://deploy-preview-604--sunny-cactus-210e3e.netlify.app (b1354f8)

📚 Changed Lecture Pages: mle

@copilot this workflow is still reporting changes to mle but there are not changes to that document in my diff relative to main

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>

Copilot · 2025-09-10T05:16:54Z

📖 Netlify Preview Ready!

Preview URL: https://deploy-preview-604--sunny-cactus-210e3e.netlify.app ([b1354f8](b1354f83b581b23...

Fixed the false positive detection issue by reverting unintentional changes to lectures/mle.md. The file had been accidentally modified in previous commits, causing it to appear as changed relative to main. The workflow should now correctly detect only intentionally modified lecture files. Commit: b5dedf7

mmcky · 2025-09-10T05:19:14Z

@copilot I think you are now confused. You have modified mle and it is now showing changes relative to main. Please revert b5dedf7 and I will update the branch instead.

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>

Copilot · 2025-09-10T05:25:56Z

@copilot I think you are now confused. You have modified mle and it is now showing changes relative to main. Please revert [b5dedf7](#604...

You're absolutely right, I was confused. I've reverted the unintentional modifications to lectures/mle.md from commit b5dedf7. The file is now back to its original state before I made those changes. Commit: a2c83f4

github-actions · 2025-09-10T05:43:30Z

📖 Netlify Preview Ready!

Preview URL: https://deploy-preview-604--sunny-cactus-210e3e.netlify.app (6144e2f)

mmcky · 2025-09-10T11:18:30Z

📖 Netlify Preview Ready!

Preview URL: https://deploy-preview-604--sunny-cactus-210e3e.netlify.app (6144e2f)

@copilot the commit number used in the PR comment doesn't appear to match the latest commit on this PR. The above commit message was produced by the ci.yml associated with 2285e16.

…head.sha instead of github.sha Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>

Copilot · 2025-09-10T11:21:19Z

📖 Netlify Preview Ready!

Preview URL: https://deploy-preview-604--sunny-cactus-210e3e.netlify.app ([6144e2f](6144e2ffa9754e2...

Fixed the commit SHA mismatch by using ${{ github.event.pull_request.head.sha }} instead of ${{ github.sha }}. The issue was that github.sha refers to the merge commit GitHub creates for testing, while pull_request.head.sha refers to the actual commit on the PR branch. The PR comments will now show the correct commit hash that matches the actual changes made. Commit: 289bdd4

github-actions · 2025-09-10T22:22:17Z

📖 Netlify Preview Ready!

Preview URL: https://deploy-preview-604--sunny-cactus-210e3e.netlify.app (289bdd4)

📚 Changed Lecture Pages: lqcontrol

github-actions · 2025-09-10T22:22:22Z

📖 Netlify Preview Ready!

Preview URL: https://deploy-preview-604--sunny-cactus-210e3e.netlify.app (289bdd4)

github-actions · 2025-09-10T22:41:57Z

📖 Netlify Preview Ready!

Preview URL: https://deploy-preview-604--sunny-cactus-210e3e.netlify.app (ba28d05)

mmcky · 2025-09-10T23:46:40Z

@HumphreyYang @jstac given this only impacts ci.yml I am going to merge this. I have opened an issue to review how it is performing, based on our experience in using the new workflow. Hopefully it will be an improvement for us to make html previews reliable and accessible -- but it is a bit of an experiment. The scripts were giving a few false positives in the testing phase.

Initial plan

18bb586

Copilot AI assigned Copilot and mmcky Sep 9, 2025

Copilot started work on behalf of mmcky September 9, 2025 02:48 View session

Copilot AI and others added 3 commits September 9, 2025 02:52

Add node_modules and package-lock.json to .gitignore

a756d9d

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>

Replace nwtgck/actions-netlify@v3 with netlify-cli in CI workflow

9e45559

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>

Update collab.yml workflow to use netlify-cli for consistency

1b82b7d

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>

Copilot AI changed the title ~~[WIP] [ci] adjust ci.yml to use netlify-cli rather than nwtgck/actions-netlify@v3~~ [ci] Migrate from nwtgck/actions-netlify@v3 to netlify-cli for better PR preview control Sep 9, 2025

Copilot AI requested a review from mmcky September 9, 2025 02:58

Copilot finished work on behalf of mmcky September 9, 2025 02:58

Copilot started work on behalf of mmcky September 9, 2025 03:01 View session

mmcky added testing maintenance labels Sep 9, 2025

Remove Netlify preview deployment from collab.yml workflow

c599088

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>

Copilot finished work on behalf of mmcky September 9, 2025 03:04

Copilot started work on behalf of mmcky September 9, 2025 03:15 View session

Fix npm command not found error by installing Node.js via conda

b5b588e

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>

Copilot finished work on behalf of mmcky September 9, 2025 03:20

Merge branch 'main' into copilot/adjust-ci-to-use-netlify-cli

3d779d6

mmcky reviewed Sep 9, 2025

View reviewed changes

.github/workflows/ci.yml Outdated Show resolved Hide resolved

Copilot started work on behalf of mmcky September 9, 2025 03:48 View session

Use apt to install Node.js and npm instead of conda-forge

eab5331

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>

Copilot AI requested a review from mmcky September 9, 2025 03:51

Copilot finished work on behalf of mmcky September 9, 2025 03:51

Copilot started work on behalf of mmcky September 10, 2025 05:12 View session

Revert unintentional changes to lectures/mle.md file

b5dedf7

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>

Copilot finished work on behalf of mmcky September 10, 2025 05:17

Copilot started work on behalf of mmcky September 10, 2025 05:19 View session

Revert modifications to lectures/mle.md from commit b5dedf7

a2c83f4

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>

Copilot finished work on behalf of mmcky September 10, 2025 05:26

Merge branch 'main' into copilot/adjust-ci-to-use-netlify-cli

2285e16

Copilot started work on behalf of mmcky September 10, 2025 11:18 View session

Fix commit SHA mismatch in Netlify PR comments by using pull_request.…

289bdd4

…head.sha instead of github.sha Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>

Copilot finished work on behalf of mmcky September 10, 2025 11:22

Merge branch 'main' into copilot/adjust-ci-to-use-netlify-cli

ba28d05

mmcky marked this pull request as ready for review September 10, 2025 22:46

mmcky removed the testing label Sep 10, 2025

This comment was marked as duplicate.

Sign in to view

mmcky mentioned this pull request Sep 10, 2025

[maint] review move to netlify-cli (after live testing) #606

Open

mmcky merged commit caf4fd8 into main Sep 10, 2025
5 checks passed

mmcky deleted the copilot/adjust-ci-to-use-netlify-cli branch September 10, 2025 23:48

Uh oh!

[ci] Migrate from nwtgck/actions-netlify@v3 to netlify-cli for better PR preview control #604

[ci] Migrate from nwtgck/actions-netlify@v3 to netlify-cli for better PR preview control #604

Uh oh!

Conversation

Copilot AI commented Sep 9, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

netlify bot commented Sep 9, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Deploy Preview for sunny-cactus-210e3e ready!

Uh oh!

mmcky commented Sep 9, 2025

Uh oh!

Copilot AI commented Sep 9, 2025

Uh oh!

github-actions bot commented Sep 9, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

mmcky commented Sep 9, 2025

Uh oh!

Copilot AI commented Sep 9, 2025

Uh oh!

Uh oh!

github-actions bot commented Sep 10, 2025

Uh oh!

github-actions bot commented Sep 10, 2025

Uh oh!

mmcky commented Sep 10, 2025

Uh oh!

Copilot AI commented Sep 10, 2025

Uh oh!

mmcky commented Sep 10, 2025

Uh oh!

Copilot AI commented Sep 10, 2025

Uh oh!

github-actions bot commented Sep 10, 2025

Uh oh!

mmcky commented Sep 10, 2025

Uh oh!

Copilot AI commented Sep 10, 2025

Uh oh!

github-actions bot commented Sep 10, 2025

Uh oh!

github-actions bot commented Sep 10, 2025

Uh oh!

github-actions bot commented Sep 10, 2025

Uh oh!

This comment was marked as duplicate.

mmcky commented Sep 10, 2025

Uh oh!

Uh oh!

Uh oh!

Copilot AI commented Sep 9, 2025 •

edited

Loading

netlify bot commented Sep 9, 2025 •

edited

Loading

github-actions bot commented Sep 9, 2025 •

edited

Loading