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

Packaging for composer / packagist #11

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

joostfaassen
Copy link

Hi @albeebe, nice work on this project!

I hope you'll consider this PR which enables your library to be used through composer and be submitted to packagist.org. This way other projects can more easily use your code in their projects.

Changes:

  • Added composer.json
  • Split the 3 classes from class.findmyiphone.php into src/ with FindMyiPhone as their namespace
  • Updated example.php and README.md to incorporate the previous changes.
  • Added a .editorconfig file to follow the current white-space conventions (tabs vs spaces)
  • Added .gitignore to play nice with composer

I've tested and can confirm that the example works as before.

If you choose to accept this PR, please go to packagist.org and submit this package. This way other projects can pull in this code through composer.

Let me know if there's any suggestions or questions!
Joost

@MrLuit
Copy link

MrLuit commented Apr 22, 2017

Why use composer instead of just one class file? I really don't see how this would be useful. It's only one simple file.

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.

2 participants