Skip to content

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