Move git calls to /admin route
#953
Merged
Conversation
This file contains 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
* Make calls explicit to minimize possible accidental server trip during certain conditions. I would have combined these but with the server hit in OpenUserJS#944 we are teetering with ENOMEM failures * Stray commas removed * Make some of the output pretty ... inversely related to OpenUserJS#37 * Reorder /admin route on most likely utilized FIFO * Move clone identify to /admin tools as well **NOTE** This should fix the main /about route from 500 Applies to OpenUserJS#944
Martii
added
enhancement
Yes it does (at least right now)... added bug label |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
NOTE
This should fix the main /about route from 500
Applies to #944