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

Mutagen + code improvements #81

Merged
merged 46 commits into from
May 6, 2020
Merged

Mutagen + code improvements #81

merged 46 commits into from
May 6, 2020

Conversation

alfredsgenkins
Copy link
Contributor

@alfredsgenkins alfredsgenkins commented May 6, 2020

What is new:

  • Mutagen support, allows to improve Mac performance
  • You can run frontend container on the first install - the popular issue is now eliminated
  • No more code duplications. One start script for most cases. Accepts options.
  • Theme is no longer build in the app container if frontend container is in the stack
  • Proper sequences in docker-compose files: depends_on and wait-for-it
  • Bump of docker-compose version to 3.6

- Add volume core-frontend-data for core frontend container
- Implement mutagen configuration for both core and frontend dc configs
- FIx node container not having wget
- Keep maildev configuration in local setup only
Make them be mounted into frontend container on deploy
instead of copied on build
Optimize sync making it two-way-resolved
Propagate www-data as default file owner
to corresponding compose file
- remove nginx session by introducing named volume
- extend mutagen permission config with default file mode
to resolve container naming issue in yml files
Move dependency installation to build script
Replace deploy script with single command
Remove core-frontend-data volume, mount app-data directly
Rename start-with-frontend script
Remove waiting for composer.json files in main script
@alfredsgenkins alfredsgenkins merged commit ec901c9 into 2.x-stable May 6, 2020
@alfredsgenkins alfredsgenkins deleted the 2.x-mutagen branch October 15, 2020 06:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants