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

Update to 1.13 #2

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Update to 1.13 #2

wants to merge 12 commits into from

Conversation

poma123
Copy link

@poma123 poma123 commented Apr 5, 2019

It's compatible with 1.13.x (tested), 1.12.x (tested), 1.11 (haven't tested yet) and 1.10 (haven't tested yet).

poma123 added 7 commits April 3, 2019 20:35
- NextStopBroadcastEvent (when the plugin sends the next stop message)
- StationReachEvent (when a player reaches a station)
- TerminusEvent (when a terminus station ejects the player)
@Slaymd
Copy link
Owner

Slaymd commented Apr 5, 2019

Really nice! thank you I'll test that soon. ;)

@tkf0
Copy link

tkf0 commented Apr 11, 2019

Thank you for working on 1.13 compat! This plugin is of very high relevance for my entire Minecraft world infrastructure :-)

One thing I noticed is that you included IDE (e.g. the .idea/ folder) and binary (the compiled JAR) files. It is perhaps better to not include them in the patch and add them to .gitignore instead.

@tkf0
Copy link

tkf0 commented Jun 9, 2019

@poma123 Hey! I noticed your patch still includes the aforementioned files.
Here's how to fix that: http://www.codeblocq.com/2016/01/Untrack-files-already-added-to-git-repository-based-on-gitignore/

Also, do your changes work with version 1.14?

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.

3 participants