php-resque-scheduler icon indicating copy to clipboard operation
php-resque-scheduler copied to clipboard

Project refactored PSR1/2/4, OOP, PHP 5.6+ complement framework yii2

Open spiritdead opened this issue 7 years ago • 8 comments

tonight i will push a couple of improvements, but this is my refactor, i rewrite this completely from the beginning and i fix a couple of bugs from the old version

https://github.com/spiritdead/resque + plugin php-resque-scheduler

https://github.com/spiritdead/yii2-resque

if someone want to help me improving this welcome

spiritdead avatar Apr 27 '17 06:04 spiritdead

https://github.com/spiritdead/resque/blob/master/composer.json#L13-L18

You've wiped the history of the repo and removed the original author from composer.json.

If this is a refactoring, why did you deliberately claim everything to yourself?

If this is a complete rewrite, why is there code from this repo in your repo without credit?

It's good that someone is working on the code, but I don't think that this is the way to go.

firoxer avatar Apr 27 '17 07:04 firoxer

@firoxer this is a refactor, this no have any documentation, readme, nothing, i need to add all of these things, if you can provide me i can add in the next commit, i put that for my package

spiritdead avatar Apr 27 '17 07:04 spiritdead

You didn't really answer the question...

firoxer avatar Apr 27 '17 07:04 firoxer

@firoxer i forgot, i'm telling you, i'm not in my house to refresh the composer, and i have to push some changes too

spiritdead avatar Apr 27 '17 07:04 spiritdead

@firoxer for the version 1.0.4 i added a special thanks

spiritdead avatar May 20 '17 09:05 spiritdead

In my opinion "inspired by X" is insufficient when most of the code is not yours but copied and pasted.

firoxer avatar May 20 '17 10:05 firoxer

you can see the repo, the code is completly different, but i have to improve the Polymorphism and base classes

spiritdead avatar May 20 '17 10:05 spiritdead

You seem to be unable to see the problem. This conversation is pointless.

firoxer avatar May 20 '17 10:05 firoxer