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

Readyyy!!! #253

Merged
merged 3 commits into from
Jun 5, 2017
Merged

Readyyy!!! #253

merged 3 commits into from
Jun 5, 2017

Conversation

kyranet
Copy link
Contributor

@kyranet kyranet commented Jun 5, 2017

Proposed Semver Increment Bump: [MAJOR/MINOR/PATCH]

minor

Changes Proposed in this Pull Request (List new items in CHANGELOG.MD)

  • Hey owner, I'm ready!

(If Applicable) What Issue does it fix?

Fixes...

@@ -130,6 +130,7 @@ module.exports = class Komada extends Discord.Client {
}));
await this.configuration.initialize(this);
this.ready = true;
this.emit("log", `Ready to serve ${this.guilds.size} guilds.`);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Needs to be more fancy. 1000x fancier for Komada.

Failure to do so will be 1000 years in the slow lane, no appeal.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have an idea, let's attach this log with a rocket.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The heck, why not customizable with komada.configs.readyMessage?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Do eet.

@CyberiumShadow
Copy link
Collaborator

Komada Ready for Combat!

@CyberiumShadow CyberiumShadow merged commit 3e011b0 into indev Jun 5, 2017
@CyberiumShadow CyberiumShadow deleted the indev-ready branch June 5, 2017 08:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants