Skip to content

2.x WIP (don't merge yet) #447

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

Jump to bottom
Merged
merged 54 commits into from
Feb 7, 2023
Merged

2.x WIP (don't merge yet) #447

merged 54 commits into from
Feb 7, 2023

Conversation

@cweagans
Copy link
Owner

@cweagans cweagans commented Feb 3, 2023

commit messages are probably not going to be great on this PR.

Putting this all in a branch to cut down on email notifications about CI breaking.

@coveralls
Copy link

coveralls commented Feb 3, 2023 •
edited
Loading

Coverage Status

Coverage: 80.0% (+40.06%) from 39.937% when pulling eaa8ee7 on 2.x-wip into a4b5731 on master.

@cweagans cweagans changed the title 2.x WIP 2.x WIP (don't merge yet) Feb 3, 2023
@cweagans cweagans mentioned this pull request Feb 3, 2023
Closed
@cweagans cweagans mentioned this pull request Feb 3, 2023
Closed
cweagans added 22 commits February 2, 2023 18:08
@cweagans
Add patch for test
13202a8
@cweagans
Remove old fixtures dir
b31ba98
@cweagans
Add runComposerInstall action
f88e6b9
@cweagans
Add package patch depth override list
32c9100
@cweagans
Misc changes to PatchDownloader
39478cb
@cweagans
wip
d4d513a
@cweagans
Just use the project root as the plugin - no need to symlink stuff in…
2419da6 
…to place
@cweagans
Test the availability and function of patch tools
1c90157
@cweagans
exclude tests/* from coverage reporting - changes to tests made phpst…
5f944b5 
…orm report this dir for some reason
@cweagans
Add an extra field to patch object for people extending the plugin
8ef28ee
@cweagans
Instead of passing url and description to events, pass the Patch object
3d3cfe3
@cweagans
Clean up output a bit
d52268e
@cweagans
Add support for debugging individual acceptance tests
d802f8c
@cweagans
Load composer-patches as early as possible
f6dea08
@cweagans
Move PatchEvent(s) and add PluginEvent(s)
97c6013
@cweagans
Rename PatchDownloader to Downloader
5505a00
@cweagans
Rename PatchLoader to Resolver
f95a58d
@cweagans
Change PatcherInterface to return a bool from apply()
9fe7bf4
@cweagans
More event rename stuff
b2e5324
@cweagans
Add Patcher
c549d57
@cweagans
Add PluginEventTest
4c71425
@cweagans
After dispatching event, use the capabilities that are in the event (…
8421317 
…in case somebody needs to alter them)
This was referenced Feb 6, 2023
Closed
Closed
Closed
Closed
cweagans added 20 commits February 6, 2023 14:49
@cweagans
Use associative arrays under the hood for compat with JsonFile
679106e
@cweagans
Install patches from a lock file when possible
816a6af
@cweagans
Run Composer in-process, rather than starting a subprocess
04f2585
@cweagans
Remove commented code
9df6165
@cweagans
Clean up patches.lock files after tests
807df99
@cweagans
Test applying a patch using Git
5bafa17
@cweagans
Move composer test code to a codeception helper
23341c6
@cweagans
Test behavior when no patchers are available
eac05c1
@cweagans
Exclude a bunch of stuff from dist packages
ddf5d85
@cweagans
Make sure it's possible to see the full composer output
eb18fea
@cweagans
Test that the plugin can be enabled and unconfigured without issues
2dcfb3f
@cweagans
Add clearPatchesForPackage method to patchcollection
ab3ae5f
@cweagans
Remove outdated comment
6dd9fcb
@cweagans
Add getPatchedPackages method
cfc9515
@cweagans
Cache downloaded patches
643e86d
@cweagans
Fix indentation
49fccdf
@cweagans
Remove package object from PatchEvent + add another event
c3f4e25
@cweagans
Add repatch and relock commands
b08efcc
@cweagans
Args should be optional
5ecb0d9
@cweagans
Add basic tests for commands
eaa8ee7
@cweagans
Copy link
Owner Author

cweagans commented Feb 7, 2023

Gonna merge this and then continue on another branch shortly.

@cweagans cweagans merged commit e95b8e6 into master Feb 7, 2023
@cweagans cweagans deleted the 2.x-wip branch February 7, 2023 18:40
@cweagans
Copy link
Owner Author

cweagans commented Feb 8, 2023

For anyone coming here from various issues, the 2.0.0 todo list is here: #451

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@arcanumbridge arcanumbridge arcanumbridge left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@cweagans @coveralls @joejoseph00 @arcanumbridge @cafuego