Align Agoric docs integration with latest SDK and stabilize CI Node versioning#1298
Conversation
Co-authored-by: turadg <21505+turadg@users.noreply.github.com>
Deploying documentation with
|
| Latest commit: |
17f0d8d
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://e95daf39.documentation-7tp.pages.dev |
| Branch Preview URL: | https://copilot-update-agoric-sdk-in.documentation-7tp.pages.dev |
|
Cloudflare deployment logs are available here |
|
@copilot also:
|
|
@copilot CI is failing now on |
... Adjusted the getting-started workflow install step to skip Puppeteer/Cypress downloads (PUPPETEER_SKIP_DOWNLOAD=1, CYPRESS_INSTALL_BINARY=0), matching a successful local Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot, can you please upgrade better-sqlite3 to at least |
|
@copilot, I still see Can you revert your previous change, and instead put |
Reverted the devDependency and added a package.json resolution for better-sqlite3 (^10.0.1), then regenerated yarn.lock with yarn install. Commit: 3e26c6f.
Updated the resolution for better-sqlite3 to ^10.0.1, regenerated yarn.lock via yarn install, and ran yarn dedupe to update the lockfile. Commit: 4cde4ad. |
|
@copilot, the change in the CI job Please abstract this so that all the matrix strategies specify |
Head branch was pushed to by a user without write access
|
The failing tests for getting_started isn't required so we could clean up and merge this. PR to fix getting_started: |
Sounds good! Can I punt it back to you? Added to my review queue. |
Co-authored-by: turadg <21505+turadg@users.noreply.github.com>
Co-Authored-By: turadg <21505+turadg@users.noreply.github.com>
Co-authored-by: turadg <21505+turadg@users.noreply.github.com>
Co-authored-by: michaelfig <457244+michaelfig@users.noreply.github.com>
Co-authored-by: michaelfig <457244+michaelfig@users.noreply.github.com>
Co-Authored-By: michaelfig <457244+michaelfig@users.noreply.github.com>
Co-authored-by: michaelfig <457244+michaelfig@users.noreply.github.com>
Co-authored-by: michaelfig <457244+michaelfig@users.noreply.github.com>
Co-authored-by: michaelfig <457244+michaelfig@users.noreply.github.com>
ae832b7 to
17f0d8d
Compare
turadg
left a comment
There was a problem hiding this comment.
I'm not sure the NODE_NEW_VERSION is worth the abstraction. We could just use current LTS with lts. Not a blocker
|
@michaelfig I cleaned up the commits and reviewed, but GH settings won't let me merge:
We could change the setting but I think it's worth keeping for now. PTAL. |
Sounds good. I reviewed, and now it's merged! |
The agoric-sdk integration tests were failing due to removed
exports.jstype re-exports and stale @Agoric dev dependencies. CI Node versioning also needed a stable alias to avoid required-check name churn.Dependency alignment
@agoric/*packages to current dev tags.better-sqlite3resolution override and refreshed the lockfile (including dedupe) to avoid older transitive builds.Explicit type imports
@agoric/zoe/exported.jsusage with JSDoc@importsyntax and grouped imports into single JSDoc blocks.@agoric/zoe,@agoric/ertp,@agoric/internal).eslint-plugin-jsdocand disabledjsdoc/tag-linesto support the new JSDoc import style.Zoe contract updates
atomicRearrangeusage tozcf.atomicRearrange.CI Node aliasing
NODE_NEW_VERSIONworkflow env and mapped thenode-newmatrix value through it.Example:
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.