This repository was archived by the owner on Apr 27, 2019. It is now read-only.
Development#124
Merged
kharidiron merged 37 commits intoStarryPy:developmentfrom Feb 7, 2015
Merged
Conversation
…guesswork going on right now...
…n. added own comment directives in per-packet to help keep track.
…erly correct these latter.
…k-a-mole on plugin errors, namely warp.
…fixed client crash on first connect to server (resulting from give item)
…mand out of command dispatecher, and into warpy. Ship warping still DOES NOT WORK. Packet parsing issues remain to be sorted out.
… bugging out over)
I removed the extra logging-level that was put in (TRACE), and instead pushed everything onto DEBUG. In addition, I modified the default options the live in the config.json file. My reasoning is that, logging was somewhat confusing, and the way it was setup carried more options than were needed. Instead, the server-owner need-only choose their logging-level from the config file, and run the server. In turn, the server now pushes all logging information into one central log-file, to reduce redundancy. I will likely add another log level back in for packet-dumping, since that is sometimes more than what debug needs as well.
…nvenince command based on the PoI system. Ship-to-ship warping does not work currently. My guess is that, reading celestial coordinates from a player's protocol is currently not working as expected. This is less imporant for production, so I'll fix it whenever I get to it.
… to see my test PoI's and bookmarks
…arding 'local' chat skipping IRC. Added uptime plugin
…eeded). Updated config.json.defaults.
… warp packets. Eveything else worked.
Contributor
There was a problem hiding this comment.
if not data:, wouldn't be better?
Contributor
|
After all, 👍. Great that code is working with newest version of Starbound! \o/ |
Member
Author
|
After a bit of discussion in the IRC, we are going to stick with the use of |
kharidiron
added a commit
that referenced
this pull request
Feb 7, 2015
Merging changes in. Soon on to master!
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 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.
All my changes to-date.