Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Performance & Standardization #34

Closed
wants to merge 19 commits into from
Closed

Conversation

zenware
Copy link
Contributor

@zenware zenware commented Mar 28, 2013

I modified some of the code for performance and PSR-1 and 2 standards T3charmy added the shell script and made several modifications himself. This version I'm pushing is fully compatible with your current version but should run more smoothly (compatibility and other testing was done by T3charmy) I don't actually play this game I just felt like improving the code base for fun and for a friend.

Also things like the MIT License are actually what is called copyleft, and are intended to remove the sole copyright privileged held by the original author. So I removed the "Copyright" you pasted into the MIT License, though that's quite irrelevant and you can keep it if you want.

@zenware
Copy link
Contributor Author

zenware commented Mar 28, 2013

I may have left out some files in the rush.

@ghost ghost assigned Dygear Mar 28, 2013
@zenware
Copy link
Contributor Author

zenware commented Apr 4, 2013

Fuck, sorry man I messed up originally and had some weird incorrect commits because it was 3am each night for the first few commits, and then I switched out all my progress and moved to something that would be an improvement to what was currently being run without breaking compatibility and it did screw up the metric, I apologize, I'll see if I can't clear it up.

@zenware zenware closed this Apr 4, 2013
@Dygear
Copy link
Owner

Dygear commented Apr 4, 2013

Thank you for the work, it's a great upgrade to the codebase. I can't wait to see the next pull request from you, I'm sure that will be awesome.

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.

None yet

2 participants