Skip to content
This repository has been archived by the owner on Dec 2, 2018. It is now read-only.

Added Updater #17

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

Added Updater #17

wants to merge 10 commits into from

Conversation

Ayumi0
Copy link
Collaborator

@Ayumi0 Ayumi0 commented Sep 29, 2018

Added Updater.

@crejk crejk added the enhancement New feature or request label Sep 29, 2018
@crejk
Copy link
Contributor

crejk commented Sep 29, 2018

Please use java 8. We don't support higher versions.

@crejk
Copy link
Contributor

crejk commented Sep 30, 2018

Our proposal:
From the txt file, go to json.
For example:

[
    {
        "version": "1.0.0",
        "release_date": "2018-09-30",
        "priority": "standard",
        "changelog": [
            "Changed A",
            "Changed B",
            "Changed C",
            "Changed D"
        ]
    },
    {
        "version": "1.0.1",
        "release_date": "2018-10-01",
        "priority": "high",
        "changelog": [
            "Changed A",
            "Changed B",
            "Changed C",
            "Changed D"
        ]
    }
]

"Added version checker"
]
}
]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

endline

@@ -12,4 +12,4 @@ private ChatColorUtil() {
public static String colored(@NotNull String content) {
return ChatColor.translateAlternateColorCodes('&', content);
}
}
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

endline

Copy link
Contributor

@crejk crejk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, thanks.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants