chore: sync CODEOWNERS-protected infrastructure files from develop #710
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
MiguelLZPF
added a commit
that referenced
this pull request
Nov 25, 2025
- Add missing import of _TIME_TRAVEL_RESOLVER_KEY to TimeTravel.sol Solidity file-level constants are not inherited; must be imported explicitly - Add @nestjs/cli to mass-payout SDK devDependencies Required for 'nest build' command to function These pre-existing issues were exposed when CI ran comprehensive tests on PRs #710 and #711. Signed-off-by: Miguel_LZPF <[email protected]>
…main Updates CODEOWNERS-protected CI/CD files from develop branch: Workflow files (8 updated/new): - ats.publish.yml: Added ref input, fixed release tag matching, improved Node version and access flags - ats.release.yml: Added release authorization checks, enhanced validation and changelog generation - ats.test.yml: Updated to Node 22.x - changeset-check.yml: Enhanced changeset validation logic - mp.publish.yml: Synchronized with ATS improvements - mp.release.yml: Updated release workflow - mp.test.yml: Node version updates - workflows/README.md: New comprehensive CI/CD documentation Configuration files (1 updated): - .gitignore: Updated patterns Part 1/2 of develop->main sync (infrastructure changes only) Signed-off-by: Miguel_LZPF <[email protected]>
MiguelLZPF
force-pushed
the
feat/infrastructure-sync-from-develop
branch
from
48ab0d4 to
12ff4f8
Compare
MiguelLZPF
added a commit
that referenced
this pull request
Nov 25, 2025
- Add missing import of _TIME_TRAVEL_RESOLVER_KEY to TimeTravel.sol Solidity file-level constants are not inherited; must be imported explicitly - Add @nestjs/cli to mass-payout SDK devDependencies Required for 'nest build' command to function These pre-existing issues were exposed when CI ran comprehensive tests on PRs #710 and #711. Signed-off-by: Miguel_LZPF <[email protected]>
Contributor
Author
|
Closing to recreate from current develop branch. PR #710 passes all tests, but recreating both PRs from the same develop snapshot ensures consistency and maintains the parallel review design. New PR will be created shortly with identical changes from up-to-date develop. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Sync PR: Aligns CODEOWNERS-protected infrastructure files between develop and main branches to prepare for develop→main merge.
Important Context: Most changes in this PR were already approved and exist on main but have diverged from develop. This sync ensures both branches have consistent infrastructure for the upcoming full merge.
New Changes Only (not yet on main):
Files Updated (9 CODEOWNERS-protected files):
Why This PR:
Type of change
Testing
Verification:
No code logic changes - only workflow configuration alignment.
Node version: N/A (infrastructure only)
Checklist
Related: This PR should merge in parallel with PR #2 (codebase sync). Both are required for complete develop→main alignment.