Warn on the use of console

[ashur]

Summary

• Adds no-console rule as warn, warning on use of console in all cases
• Adds a CHANGELOG.md and a pull request template to the project

Adding no-console as a warning could help catch any debugging uses of console calls before being committed or pushed.

Usage

In cases where a console call is desired in non-debugging contexts, the rule can be disabled:

// eslint-disable-next-line no-console
console.log("This message should be logged to an end-user's browser");

Tasks

• [x] Update CHANGELOG

Notes

ESLint on console use:

In JavaScript that is designed to be executed in the browser, it’s considered a best practice to avoid using methods on console. Such messages are considered to be for debugging purposes and therefore not suitable to ship to the client.

Links

• https://eslint.org/docs/latest/rules/no-console

[jakefentress]

@ashur I like the no-console for commits, but this might keep things from compiling locally so you couldn't use a console.log for debugging, either. But, that may be more a function of how we're currently using eslint in the bootstrap as opposed to maybe how it should be used.

[ashur]

@jacobfentress Great point! I dropped the rule to warn instead of error to prevent breaking existing uses of eslint-config-fk.