updated: eslint setup

Author: GianlucaGuarini

.eslintignore

@@ -0,0 +1,2 @@
+import config from '@riotjs/prettier-config'
+export default config

.eslintrc

@@ -3,7 +3,7 @@ import { dashToCamelCase } from './strings.js'
 /**
  * Get all the element attributes as object
  * @param   {HTMLElement} element - DOM node we want to parse
- * @returns {Object} all the attributes found as a key value pairs
+ * @returns {object} all the attributes found as a key value pairs
  */
 export function DOMattributesToObject(element) {
   return Array.from(element.attributes).reduce((acc, attribute) => {

.prettierrc.cjs

@@ -0,0 +1,12 @@
+import { defineConfig, globalIgnores } from 'eslint/config'
+import riotEslintConfig from 'eslint-config-riot'
+
+export default defineConfig([
+  globalIgnores(['*.cjs']),
+  { extends: [riotEslintConfig] },
+  {
+    rules: {
+      'fp/no-mutating-methods': 0,
+    },
+  },
+])

.prettierrc.js

@@ -7,9 +7,9 @@ export function noop() {
 
 /**
  * Autobind the methods of a source object to itself
- * @param   {Object} source - probably a riot tag instance
+ * @param   {object} source - probably a riot tag instance
  * @param   {Array<string>} methods - list of the methods to autobind
- * @returns {Object} the original object received
+ * @returns {object} the original object received
  */
 export function autobindMethods(source, methods) {
   methods.forEach((method) => {

dom.js

@@ -30,8 +30,8 @@ export function memoize(fn) {
 /**
  * Generate key-value pairs from a list of attributes
  * @param   {Array} attributes - list of attributes generated by the riot compiler, each containing type, name, and evaluate function
- * @param   {Object} scope - the scope in which the attribute values will be evaluated
- * @returns {Object} An object containing key-value pairs representing the computed attribute values
+ * @param   {object} scope - the scope in which the attribute values will be evaluated
+ * @returns {object} An object containing key-value pairs representing the computed attribute values
  */
 export function generatePropsFromAttributes(attributes, scope) {
   return attributes.reduce((acc, { type, name, evaluate }) => {

eslint.config.js

@@ -2,32 +2,30 @@ import { isObject } from './checks.js'
 
 /**
  * Helper function to set an immutable property
- * @param   {Object} source - object where the new property will be set
+ * @param   {object} source - object where the new property will be set
  * @param   {string} key - object key where the new property will be stored
  * @param   {*} value - value of the new property
- * @param   {Object} options - set the property overriding the default options
- * @returns {Object} - the original object modified
+ * @param   {object} options - set the property overriding the default options
+ * @returns {object} - the original object modified
  */
 export function defineProperty(source, key, value, options = {}) {
-  /* eslint-disable fp/no-mutating-methods */
   Object.defineProperty(source, key, {
     value,
     enumerable: false,
     writable: false,
     configurable: true,
     ...options,
   })
-  /* eslint-enable fp/no-mutating-methods */
 
   return source
 }
 
 /**
  * Define multiple properties on a target object
- * @param   {Object} source - object where the new properties will be set
- * @param   {Object} properties - object containing as key pair the key + value properties
- * @param   {Object} options - set the property overriding the default options
- * @returns {Object} the original object modified
+ * @param   {object} source - object where the new properties will be set
+ * @param   {object} properties - object containing as key pair the key + value properties
+ * @param   {object} options - set the property overriding the default options
+ * @returns {object} the original object modified
  */
 export function defineProperties(source, properties, options) {
   Object.entries(properties).forEach(([key, value]) => {
@@ -39,9 +37,9 @@ export function defineProperties(source, properties, options) {
 
 /**
  * Define default properties if they don't exist on the source object
- * @param   {Object} source - object that will receive the default properties
- * @param   {Object} defaults - object containing additional optional keys
- * @returns {Object} the original object received enhanced
+ * @param   {object} source - object that will receive the default properties
+ * @param   {object} defaults - object containing additional optional keys
+ * @returns {object} the original object received enhanced
  */
 export function defineDefaults(source, defaults) {
   Object.entries(defaults).forEach(([key, value]) => {
@@ -62,9 +60,9 @@ export function cloneDeep(source) {
 
 /**
  * Like Array.prototype.filter but for objects
- * @param {Object} source - target object
- * @param {Funciton} filter - filter function
- * @return {Object} filtered source or the original source received
+ * @param {object} source - target object
+ * @param {(key: unknown, value: unknown) => boolean} filter - filter function
+ * @returns {object} filtered source or the original source received
  */
 export function filter(source, filter) {
   return isObject(source)
@@ -76,9 +74,9 @@ export function filter(source, filter) {
 
 /**
  * Generate a new object picking only the properties from a given array
- * @param {Object} source - target object
+ * @param {object} source - target object
  * @param {Array} keys - list of keys that we want to copy over to the new object
- * @return {Object} a new object conaining only the keys that we have picked from the keys array list
+ * @returns {object} a new object conaining only the keys that we have picked from the keys array list
  */
 export function pick(source, keys) {
   return isObject(source)