fix: tweak default node/* settings

* the unpublished-require checks is broken for the way we use devDeps
* the deprecated-api check shouldn't kill use of e.g.
    require('url')...for now

lib/rules/mistakes.js

@@ -146,6 +146,11 @@ module.exports = {
 
   // See: https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/no-unresolved.md
   'import/no-unresolved': ['error', { commonjs: true, caseSensitive: true }],
+
+  // This does the same thing, but doesn't know about devDeps properly
+  // See: https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-unpublished-require.md
+  'node/no-unpublished-require': ['off'],
+
   // See: https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/no-extraneous-dependencies.md
   'import/no-extraneous-dependencies': [
     'error',

lib/rules/opinions.js

@@ -59,4 +59,7 @@ module.exports = {
 
   // See: https://eslint.org/docs/rules/object-shorthand
   'object-shorthand': 'error',
+
+  // See: https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-deprecated-api.md
+  'node/no-deprecated-api': ['warn'],
 };