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

Feature/issue 1 #4

Closed
wants to merge 4 commits into from
Closed

Conversation

owenvoke
Copy link

@owenvoke owenvoke commented Oct 24, 2017

Feel free to edit what you want from it.


96a324b and e29afc2 are more just aesthetic updates, and for use with actual Composer.

dcf3b60 resolves #1 by adding the license and tidying the Composer JSON info in the README.

4192d5e was an after thought, but looking at it I guess you may want to remove this commit. I was thinking that there's not much point in echoing the file's data if it's already in version control, but then realised that the repo may not always be synced down. I guess maybe another option would be to use curl or wget to pull the latest version of the file down. Such as:
wget https://raw.githubusercontent.com/TheAdnan/Yiinker/master/src/YiinkerCommand.php
curl -O https://raw.githubusercontent.com/TheAdnan/Yiinker/master/src/YiinkerCommand.php

@TheAdnan
Copy link
Owner

Can you push this to the hacktober branch?

@owenvoke
Copy link
Author

owenvoke commented Oct 25, 2017

Sorry, didn't know there was one. Sure 👍

Added at #6

@owenvoke owenvoke closed this Oct 25, 2017
@owenvoke owenvoke deleted the feature/issue-1 branch December 2, 2017 21:46
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.

Update README
2 participants