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

Lack of set function - porting proposal #17

Open
willypuzzle opened this issue Jul 12, 2019 · 3 comments
Open

Lack of set function - porting proposal #17

willypuzzle opened this issue Jul 12, 2019 · 3 comments

Comments

@willypuzzle
Copy link

I noted the lack of a set function (as here). I built a sort of set function in a project of mine. At the moment it works only on array, what do you think if I make a porting of my function on your project?
What are your general guidelines to make a pull request? Do you have any suggest?
Best Regards
Domenico Rizzo

@pierredup
Copy link
Member

@willypuzzle That would be great if you can create a PR.

Some useful info:

  • Delete the src/compiled.php file while creating the function and adding the unit tests
  • When you have completed the function and tests, run bin/compile before submitting the PR

Other than that, there are no other guidelines. You can create the PR and we can continue the discussion from there

@punit-kulal
Copy link

doesn't he also need to run the php-cs-fixer ??

@cube-dan
Copy link

cube-dan commented Jan 1, 2021

@pierredup, @willypuzzle,

It looks like there’s a baseSet method defined here: https://github.com/lodash-php/lodash-php/blob/master/src/internal/baseSet.php

Is this as simple as adding a set.php as a next door neighbor of get.php?

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

No branches or pull requests

4 participants