Merge pull request #8 from chris062689/master

Remove logging from triggers, log instead in server.js
This commit is contained in:
Christopher J. Gilbert 2017-03-30 21:32:42 -04:00 committed by GitHub
commit 0f33a06834
2 changed files with 3 additions and 4 deletions

View file

@ -55,7 +55,7 @@ client.on('message', message => {
return false;
}
logger.info(`${message.author.username} ${message.author} [Channel: ${message.channel}] triggered command: ${message.content}`);
logger.info(`${message.author.username} ${message.author} [Channel: ${message.channel}] executed command: ${message.content}`);
message.delete();
try {
@ -80,6 +80,7 @@ client.on('message', message => {
cachedTriggers.forEach(function(trigger) {
if (trigger.roles == undefined || findArray(message.member.roles.map(function(x) { return x.name; }), trigger.roles)) {
if (trigger.trigger(message) == true) {
logger.info(`${message.author.username} ${message.author} [Channel: ${message.channel}] triggered: ${message.content}`);
try {
trigger.execute(message);
} catch (err) { logger.error(err); }
@ -105,7 +106,7 @@ require("fs").readdirSync('./triggers/').forEach(function(file) {
// Load the trigger if it's a script.
if (path.extname(file) == '.js') {
logger.info(`Loaded trigger: ${file}`);
cachedTriggers.push(require(`./triggers/${file}`));
cachedTriggers[file] = require(`./triggers/${file}`);
}
});

View file

@ -27,8 +27,6 @@ exports.execute = function(message) {
request(url, function (error, response, body) {
if (!error && response.statusCode == 200) {
message.channel.sendMessage(`Github Pull Request: ${url}`);
} else {
logger.info(`PR #{issue} does not exist.`)
}
});