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/composer integration #253

Conversation

teiling88
Copy link
Member

Composer Integration with new test structure.
Added first test for login functionality

@teiling88 teiling88 force-pushed the feature/composer-integration branch from 880da13 to cbcdef9 Compare May 9, 2016 20:56
@MacGyver-NRW MacGyver-NRW merged commit 75a3cc0 into OpencachingDeutschland:master May 9, 2016
@teiling88 teiling88 deleted the feature/composer-integration branch May 10, 2016 14:00
@following5
Copy link
Contributor

following5 commented May 10, 2016

Nach diesem Merge ist der master-Branch nicht mehr lauffähig.

Warning: require_once(/mnt/ocde/htdocs/vendor/autoload.php): failed to open stream: No such file or directory in /mnt/ocde/htdocs/index.php on line 8

Fatal error: require_once(): Failed opening required '/mnt/ocde/htdocs/vendor/autoload.php' (include_path='.:/usr/share/pear:/usr/share/php') in /mnt/ocde/htdocs/index.php on line 8

@MacGyver-NRW
Kannst du die Änderungen bitte testen, bevor du sie in den Master übernimmst? Das ist zwingend notwendig, weil der Master als Basis für alle Weiterentwicklungen komplett funktionsfähig sein muss.

@teiling88
Copy link
Member Author

Das stimmt so nicht. Ein composer install lädt die third party libs und generiert den autloader. Dann funktioniert der branch auch.

Weitere Details kannst Du hier finden.

@following5
Copy link
Contributor

Ok. Kannst du solche Änderungen so dokumentieren, dass jeder Entwickler in der Lage ist den aktuellen Code wieder ans Laufen zu bekommen?

Ich mache mal einen Forenthread dazu auf, damit die anderen auch bescheid wissen.

@MacGyver-NRW
Copy link
Member

Keine Sorge Peter, die Dokumentation wird nicht zu kurz kommen.

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