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

Local Development Environment #4

Merged
merged 3 commits into from Mar 5, 2019
Merged

Local Development Environment #4

merged 3 commits into from Mar 5, 2019

Conversation

asheperak
Copy link
Collaborator

This PR introduces a standardized environment for local development. It also begins the process of creating a build / deployment process that is independent of the environment. Chef recipes can be used with different attributes in different environments.

Two changes in this PR affect files outside of the new .develop/ directory:
.gitignore to ignore the .vagrant/ directories (standard)
includes/functions.php to remove the requirement for SSL from the PHP code. I usually prefer to do this in the Apache VHost configuration for the site, or, even better, at the load balancer.

Please see the .develop/README.md for more information
This will allow us to use the same Chef recipes for
application deployment and server configration in any
environment
Copy link
Owner

@bradsknutson bradsknutson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm approving this, however won't be deploying to the production server. emcp.com is actually Paradigm's domain, and they host many things on this server that would break if forced to SSL - which is why this marketing site has an include file that forces SSL on our pages only. In the event the domain or server changes these would obviously be deployed.

@bradsknutson bradsknutson merged commit 5ea2987 into master Mar 5, 2019
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