Skip to content

Indev#24

Merged
ORelio merged 3 commits intoMCCTeam:masterfrom
featherbear:Indev
Nov 4, 2013
Merged

Indev#24
ORelio merged 3 commits intoMCCTeam:masterfrom
featherbear:Indev

Conversation

@featherbear
Copy link

No description provided.

@ORelio
Copy link
Member

ORelio commented Nov 4, 2013

Nice! Simple and efficient /exec command.

ORelio added a commit that referenced this pull request Nov 4, 2013
Removal of Bouncycastle leftover reference + /exec <script> command + ignore case on commands
@ORelio ORelio merged commit 37a80fb into MCCTeam:master Nov 4, 2013
@featherbear
Copy link
Author

I'd just like to say, if you do /exec nonexistent_script.txt, it will say and will hang indefinitely, so you might want to fix that, I'm not sure how. :P

@featherbear
Copy link
Author

And, you put it into master, instead of index, or is that intentional

@ORelio
Copy link
Member

ORelio commented Nov 5, 2013

Hem, that's not intentional, but apparently it's the person opening the pull request who chooses the branch to pull.

@featherbear
Copy link
Author

Ahhahaha oops :P should I change it? Since master should be for releases.

@ORelio
Copy link
Member

ORelio commented Nov 5, 2013

Not sure it can be changed once merged.

@featherbear
Copy link
Author

Well, can you like undo the changes? Then I could try pull request again

@ORelio
Copy link
Member

ORelio commented Nov 6, 2013

Pull undone by manual git reset.

@ORelio ORelio mentioned this pull request Nov 6, 2013
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.

2 participants