Skip to content
This repository has been archived by the owner on Dec 2, 2020. It is now read-only.

Add's the uopz's extension #65

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Add's the uopz's extension #65

wants to merge 6 commits into from

Conversation

alexmace
Copy link

Allows modification of the exit language construct etc.

Has a spec file.

include boxen::config

require php::config
# Require php version eg. php::5_4_10

Choose a reason for hiding this comment

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

This comment uses the old syntax. Can you please update to the new syntax or remove it to avoid confusion?

@alexmace
Copy link
Author

Could you explain what you mean?

I've been comparing the syntax here to other extensions in the directory, but can't spot what the difference is. Happy to make the changes on all three pull requests, but need a bit more guidance.

@sambauers
Copy link

From TFM:

You will need to change any code in your manifests like include PHP::5_X_X to the version 2 equivalent php::version { 5.X.X: }.

Let me know if this doesn't make sense.

@alexmace
Copy link
Author

Right, I get it now - the same syntax is in the comment for every other extension, so I couldn't see what what I'd one wrong. Back in a bit with an updated version.

@alexmace alexmace closed this May 13, 2015
@alexmace alexmace reopened this May 13, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants