Skip to content

Refactor Node.js package installers from lifecycle hooks to ExecutableResource #736

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
aaronpowell merged 9 commits into from
Jun 20, 2025
Merged

Refactor Node.js package installers from lifecycle hooks to ExecutableResource #736

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
c611b56
Initial plan for issue
Copilot Jun 17, 2025
fb1ee59
Create installer resource classes and refactor extension methods
Copilot Jun 17, 2025
1bac548
Mark lifecycle hooks as obsolete and update tests for new resource ap…
Copilot Jun 17, 2025
507380a
Add comprehensive tests for installer resources and publish mode excl…
Copilot Jun 17, 2025
0b845e5
Add integration tests and documentation for resource-based package in…
Copilot Jun 17, 2025
344300b
Refactor package installer resources to simplify constructors and rem…
aaronpowell Jun 17, 2025
25ccd59
Should be waiting for the completion of the installer, not for it to …
aaronpowell Jun 18, 2025
fec4f8d
Cleanup to fix error in yarn install log
aaronpowell Jun 18, 2025
e1e9a67
Merge branch 'main' into copilot/fix-732
aaronpowell Jun 18, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Merge branch 'main' into copilot/fix-732
  • Loading branch information
@aaronpowell
aaronpowell authored Jun 18, 2025
commit e1e9a67a55ee2b34740ccbf4cc276a29ecb07b1c

This merge commit was added into this branch cleanly.

There are no new changes to show, but you can still view the diff.