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

Unknown constant used #24

Closed
Max-Kolodezniy opened this issue Oct 6, 2016 · 2 comments
Closed

Unknown constant used #24

Max-Kolodezniy opened this issue Oct 6, 2016 · 2 comments

Comments

@Max-Kolodezniy
Copy link
Contributor

Unknown constant used as default parameter in \PhpOffice\PhpSpreadsheet\Settings::setCacheStorageMethod
public static function setCacheStorageMethod($method = CachedObjectStorageFactory::cache_in_memory, $arguments = [])

@Max-Kolodezniy
Copy link
Contributor Author

I've created pull request for this issue.
Not sure if I've done that properly. According to the documentation:

If you are going to be submitting a pull request, please fork from develop, and submit your pull request back as a fix/feature branch referencing the GitHub issue number

So, I have to create the issue, create branch named as Issue (issue ID?) and then create pull request? Right?

@PowerKiKi
Copy link
Member

What you did was fine. You don't necessarily need to create an issue, but you have to create a branch, possibly named after the issue or something understandable.

Next time, if there is an open issue then you may mention it in the commit message such as "FIX #24" so that the issue is automatically closed when we merge the PR.

Thanks for contributing.

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

No branches or pull requests

2 participants