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

initial sensiolabs/melody integration #43

Merged
merged 1 commit into from Feb 16, 2015

Conversation

staabm
Copy link
Collaborator

@staabm staabm commented Jan 26, 2015

This PR integrates melody-scripts into the php-console.

This means you can run melody scripts directly from your webbrowser.
the melody integration makes sure all required composer dependencies are downloaded and available to the code within the console.

On the first run (when dependencies are present which are not yet resolved) the php-console output panel will show what composer is doing in the background.
melody-console-initial

On any subsequent run (or run which only used already downloaded dependencies) the output looks like we are used to:
melody-console-run

//cc @lyrixx @jeremy-derusse

@staabm
Copy link
Collaborator Author

staabm commented Feb 15, 2015

@Seldaek ping

Seldaek added a commit that referenced this pull request Feb 16, 2015
initial sensiolabs/melody integration
@Seldaek Seldaek merged commit a5c4cf8 into Seldaek:master Feb 16, 2015
@Seldaek
Copy link
Owner

Seldaek commented Feb 16, 2015

Looks fine I guess, thanks :)

@staabm staabm deleted the melodyscripts branch February 16, 2015 14:34
@staabm
Copy link
Collaborator Author

staabm commented Feb 16, 2015

@Seldaek ok, just getting you general opinion on the feature was important... if any refinements are required I won't bother you and go ahead fixing them.

thanks.

@Seldaek
Copy link
Owner

Seldaek commented Feb 16, 2015

Ah right I forgot I gave you commit access :D Might be good to add a mention of how to do this in the README and or in the few samples at the bottom of the page.

@staabm
Copy link
Collaborator Author

staabm commented Feb 16, 2015

will do ;)

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.

None yet

2 participants