Topic/run after other any time scripts initial work#1
Merged
ferventcoder merged 2 commits intoMay 27, 2011
Merged
Conversation
Member
|
I am going to be adding these very soon. |
Contributor
Author
|
Excellent. We're using it at work and the mangement types are asking me, To which I reply, "no you're dependent on my custom version" :) It will be nice to get back on the mainline. Andy On Thu, May 19, 2011 at 1:49 PM, ferventcoder <
|
Member
|
Good stuff. :D |
GoogleCodeExporter
pushed a commit
to google-code-export/roundhouse
that referenced
this pull request
Mar 13, 2015
…as been added. Changes from chucknorris/roundhouse#1
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Based off my discussion with Rob today. I added a new option called runAfterOtherAnyTimeScripts which will add a directory that is run after sprocs, but before permissions.
I did verify that the convention is picked up and that the command line option does over ride the convention. I did not add tests for this, because I can't find tests for the other folders. If they are there, please just point me to them and I will take care of that right away.
I confess that I am a bit of a git and git hub novice. If I have blown some courtesy, please bear with me (and let me know how to fix it). Hopefully the code is useful enough to make it worth your while.
Andy