Remove no-promise-in-callback and promise/no-callback-in-promise …

…rules

They were more annoying than useful. At least today when you have to deal with a lot of non-Promise APIs. Some APIs will also never be Promise, like events and streams.

config/plugins.js

@@ -30,8 +30,6 @@ module.exports = {
 	rules: {
 		'no-use-extend-native/no-use-extend-native': 'error',
 		'promise/param-names': 'error',
-		'promise/no-promise-in-callback': 'error',
-		'promise/no-callback-in-promise': 'error',
 		// Enable this sometime in the future when Node.js has async/await support
 		// 'promise/prefer-await-to-then': 'error',
 		'import/default': 'error',