Logger's been set twice

[akadori]

When I looked at the code in the link, I noticed that the same process was written twice.
I wondered if this was intentional?

https://github.com/slackapi/bolt-js/blob/08b50ca75817e17849daeeafb9df519755ca49ef/src/receivers/ExpressReceiver.ts#L84:L97

    if (typeof logger !== 'undefined') {
      this.logger = logger;
    } else {
      this.logger = new ConsoleLogger();
      this.logger.setLevel(logLevel);
    }

    if (typeof logger !== 'undefined') {
      this.logger = logger;
    } else {
      this.logger = new ConsoleLogger();
      this.logger.setLevel(logLevel);
    }

[stevengill]

This sure looks like a mistake. Maybe due to a bad merge?

[filmaj]

Indeed, looks like an error during merge: aa7626b#diff-683369dc41aa82b5b3c78fe442af93e64e5ca7a02d82daf4720ba68ea2d4e0dbR83-R89

Sometimes automatic git merge combines things in ways that would be unexpected and undesirable. This is why I prefer to git rebase over git merge - even though often that's more work 🤷