Skip to content
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.

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

fix: make pathfinder composer 2.x compliant #1035

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

warlof
Copy link

@warlof warlof commented Sep 6, 2021

extract mixed class into dedicated file and address filename typo in order to make the project PSR-4 compliant and work with Composer 2

extract mixed class into dedicated file and address filename typo in order to make the project PSR-4 compliant and work with Composer 2
Copy link

@deMathias deMathias left a comment

Choose a reason for hiding this comment

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

looks good to me. Tried and no more errors.

@samoneilll
Copy link

Thanks, this should solve issues a lot of people have had with using composer 2.x, I've merged it into our fork at https://github.com/goryn-clade/pathfinder

@Xyncgas
Copy link

Xyncgas commented Oct 9, 2022

we need to merge this so new people coming from hearing about pathfinder don't get stuck running composer install

@samoneilll
Copy link

@Xyncgas this particular version of pathfinder is no longer maintained and won't work due to the very out of date Oauth implementation. It has been merged into the active fork at https://github.com/goryn-clade/pathfinder

@Xyncgas
Copy link

Xyncgas commented Nov 1, 2022

Can we bring https://github.com/goryn-clade/pathfinder to merge with this repository because this helps a lot people

@samoneilll
Copy link

We cannot because the maintainer of this repo has abandoned it.

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.

4 participants