Skip to content

Upgrade Core to 57e8c91ed68e3ee903526fd2f45cb16ca46759d8#19

Merged
jviotti merged 1 commit intomainfrom
core-static
Mar 31, 2026
Merged

Upgrade Core to 57e8c91ed68e3ee903526fd2f45cb16ca46759d8#19
jviotti merged 1 commit intomainfrom
core-static

Conversation

@jviotti
Copy link
Copy Markdown
Member

@jviotti jviotti commented Mar 31, 2026

Signed-off-by: Juan Cruz Viotti jv@jviotti.com

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 194 files

Note: This PR contains a large number of files. cubic only reviews up to 75 files per PR, so some files may not have been reviewed.

@augmentcode
Copy link
Copy Markdown

augmentcode bot commented Mar 31, 2026

🤖 Augment PR Summary

Summary: This PR upgrades the pinned sourcemeta/core dependency to commit 57e8c91ed68e3ee903526fd2f45cb16ca46759d8.

Changes:

  • Updates the Core revision recorded in DEPENDENCIES
  • Refreshes the vendored Core checkout under vendor/core to match the new pinned revision (large vendor update)
  • Keeps Codegen’s build wiring pointing at the vendored Core via cmake/FindCore.cmake

Technical Notes: This is primarily a dependency/vendor refresh; downstream behavior changes (if any) come from the Core upgrade rather than direct Codegen source changes.

🤖 Was this summary useful? React with 👍 or 👎

Copy link
Copy Markdown

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

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

Review completed. 1 suggestion posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

@@ -1,2 +1,2 @@
vendorpull https://github.com/sourcemeta/vendorpull 1dcbac42809cf87cb5b045106b863e17ad84ba02
core https://github.com/sourcemeta/core fe450b982907f99e542a0cfc78bc60d2b600ff7a
core https://github.com/sourcemeta/core 57e8c91ed68e3ee903526fd2f45cb16ca46759d8
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

DEPENDENCIES:2: Since this PR changes the pinned Core revision, consider double-checking that the vendored vendor/core tree was regenerated from the exact same commit (to avoid a hash/vendor mismatch that can break reproducible builds).

Severity: medium

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
@jviotti jviotti merged commit 8799edb into main Mar 31, 2026
12 checks passed
@jviotti jviotti deleted the core-static branch March 31, 2026 20:07
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.

1 participant