Skip to content
This repository has been archived by the owner on Jan 10, 2024. It is now read-only.

Major update #79

Open
uuf6429 opened this issue Mar 20, 2018 · 6 comments
Open

Major update #79

uuf6429 opened this issue Mar 20, 2018 · 6 comments

Comments

@uuf6429
Copy link

uuf6429 commented Mar 20, 2018

Recently, I started using this library for a project and during development, I've found how badly outdated it is. It lacks modern development practices and contains several bugs. Even worse, it seems to be abandoned, with many open issues and pull requests not being addressed at all.

Given the scale of the damage 😉, I decided to roll out my own, rewritten almost from scratch (but still sharing some common public parts).

Here's my effort and progress: https://github.com/uuf6429/Force.com-Toolkit-for-PHP/tree/Major-refactor#forcecom-toolkit-for-php

I'd be more than happy to see this merged into mainline eventually. Until then, I'd gladly listen to feedback/ideas and accept contributions (make sure you sign the SF CLA first though).

@hatzopoulos
Copy link

Nice work. I hope you can get your fixes merged in.

And I wouldn't say this project is abandoned just yet.

http://repocheck.com/#https%3A%2F%2Fgithub.com%2Fdeveloperforce%2FForce.com-Toolkit-for-PHP

image

@mwleinad
Copy link

Give @uuf6429 a cookie!
I installed your stuff and it worked correctly on the first try :) (all other libraries I tried had issues)
If developerforce is not willing to merge this up, maybe you can create a new release with the major-update branch? right now I have to specify which branch I want if I do a composer require, otherwise it install the v27 which is still an old version.

@grantjanecek
Copy link

@uuf6429 I am interested in contributing to your fork of this project.

@uuf6429
Copy link
Author

uuf6429 commented Feb 19, 2019

@hatzopoulos - haven't seen those sort of metrics before and I personally wouldn't hold on to them much - but anyway, the score went down to 3.3 now.
@mwleinad - thanks! Totally agree with what you said.
@K03413R - sure, feel free to contribute pull requests to my repo/branch and I'll merge. As time goes, I'll hand out contributor access too when it makes sense.

@DanieleGBX
Copy link

@uuf6429 Also please allow issues to be posted to your fork, so we can discuss development there.

@johnbburg
Copy link

Repo health check is now considered "In the weeds (3.3/10)." So may as well consider it all but abandoned. I'm using this on a Drupal 7 project that never had composer set up. So I'll be creating my own fork to handle PHP 7 compatibility.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants