这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@michaelshobbs
Copy link
Member

creates config:set-norestart and config:unset-norestart

I did not include these in the help output deliberately as this functionality is not compliant with Heroku functionality and is intended for internal use by things like plugins.

refs #1009

@michaelshobbs
Copy link
Member Author

ping @josegonzalez

Copy link
Member

Choose a reason for hiding this comment

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

wrong usage

@michaelshobbs
Copy link
Member Author

@josegonzalez fixed usage message

@josegonzalez
Copy link
Member

I actually thought about it more, and since it's being called by the external command, we should just use the external names (set/unset) as opposed to the actual. Can you revert that change? As well, add some docs to the plugin development section regarding it's usage.

@josegonzalez
Copy link
Member

Once ^ is done, we can merge :)

@michaelshobbs michaelshobbs force-pushed the 908_mh-configset-norestart branch from e9b489c to 6cb770b Compare March 19, 2015 14:42
@michaelshobbs
Copy link
Member Author

Done and done.

josegonzalez added a commit that referenced this pull request Mar 19, 2015
create config set/unset without restart. closes #908
@josegonzalez josegonzalez merged commit fafa33e into master Mar 19, 2015
@josegonzalez josegonzalez deleted the 908_mh-configset-norestart branch March 19, 2015 14:52
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.

3 participants