Skip to content

Adapt to use composer installer - #83

Merged
asgrim merged 25 commits into
php:mainfrom
asgrim:adapt-to-use-composer-installer
Nov 15, 2024
Merged

Adapt to use composer installer#83
asgrim merged 25 commits into
php:mainfrom
asgrim:adapt-to-use-composer-installer

Conversation

@asgrim

@asgrimasgrim commented Nov 7, 2024•
edited
Loading

Copy link
Copy Markdown
Contributor

Fixes#48

FIxes#77

@asgrimasgrim added the enhancement New feature or request label Nov 7, 2024
@asgrimasgrim added this to the 0.2.0 milestone Nov 7, 2024
@asgrimasgrim self-assigned this Nov 7, 2024
This was referenced Nov 7, 2024
Closed
Closed
TimWolla
TimWolla reviewed Nov 12, 2024
View reviewed changes
Comment threadsrc/BinaryFile.php Outdated
Comment threadsrc/BinaryFile.php Outdated
This was referenced Nov 14, 2024
Closed
Closed
Seldaek
Seldaek reviewed Nov 14, 2024
View reviewed changes
Comment threadsrc/BinaryFile.php
Seldaek
Seldaek reviewed Nov 14, 2024
View reviewed changes
Comment threadsrc/ComposerIntegration/ComposerIntegrationHandler.php
@TimWollaTimWolla mentioned this pull request Nov 14, 2024
Closed
asgrim
asgrim commented Nov 14, 2024
View reviewed changes
Comment threadsrc/ComposerIntegration/InstallAndBuildProcess.php
This was referenced Nov 15, 2024
Closed
Closed
@asgrim
asgrimforce-pushed the adapt-to-use-composer-installer branch from b28d602 to 0a9953eCompareNovember 15, 2024 10:52
@asgrim
Add require to composer.json
9c0ffe1
@asgrim
Working on getting PIE builder linked in
0737e76
@asgrim
Restructure to use Composer and PieComposerRequest to determine order…
21f1c9d
… of operations
@asgrim
Enable installs using Composer installer
d8489ab
@asgrim
Reorganise createComposer into PieComposerFactory
e8a4dd6
@asgrim
Fixed some tests after updating to use Composer
00d3164
@asgrim
Fixesphp#77- output build process progressively if -v verbose flag …
71376be
…given
@asgrim
Add listener to download Windows prebuilt binary zips
5c1618e
@asgrim
Ensure PIE working directory exists
8aca879
@asgrim
Resolve static analysis issues in Platform
1d41cda
@asgrim
Handle resultCode from Composer Installer run()
08baaa8
@asgrim
Improve logging management for Composer invokations
c68ce6d
@asgrim
Removed redundant type alias RequestedNameAndVersionPair
5bd939b
@asgrim
Added some missing internal API markers
7d37664
@asgrim
Split pie.jsons into separate paths per target PHP platform
3713a33
@asgrim
asgrimforce-pushed the adapt-to-use-composer-installer branch from 0a9953e to c935d7fCompareNovember 15, 2024 11:01
@asgrim
asgrim merged commit 76ba65b into php:mainNov 15, 2024
@asgrim
asgrim deleted the adapt-to-use-composer-installer branch November 15, 2024 11:06
@asgrimasgrim mentioned this pull request Nov 20, 2024
Closed
@asgrimasgrim mentioned this pull request Dec 3, 2024
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TimWollaTimWollaTimWolla left review comments
+1 more reviewer
@SeldaekSeldaekSeldaek left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

@asgrimasgrim

Labels

enhancementNew feature or request

Projects

None yet

Milestone

0.2.0

Development

Successfully merging this pull request may close these issues.

Optionally output build progress Investigate improving internals with Composer Installer

3 participants

@asgrim@Seldaek@TimWolla