adjust/fix/disable some prettier warnings

.eslintrc.yml

@@ -7,6 +7,7 @@ parserOptions:
   sourceType: module
 rules:
   no-console: "off"
+  no-param-reassign: "off"
   no-await-in-loop:
     - warn
   radix:

module/actor/actor-sheet.js

@@ -316,9 +316,9 @@ export default class DeltaGreenActorSheet extends ActorSheet {
     html.find(".typed-skill-add").click((event) => {
       event.preventDefault();
 
-      const targetskill = event.target.getAttribute("data-typeskill");
+      // const targetskill = event.target.getAttribute("data-typeskill");
 
-      this._showNewTypeSkillDialog(targetskill);
+      this._showNewTypeSkillDialog();
     });
 
     html.find(".typed-skill-edit").click((event) => {
@@ -643,7 +643,7 @@ export default class DeltaGreenActorSheet extends ActorSheet {
     }).render(true);
   }
 
-  _showNewTypeSkillDialog(targetskill) {
+  _showNewTypeSkillDialog() {
     let htmlContent = "";
 
     htmlContent += `<div>`;

module/deltagreen.js

@@ -98,6 +98,7 @@ Hooks.on("renderSidebarTab", async (app, html) => {
  * We use this hook to translate the sample Typed Skill
  * Note - this event is fired on only the client who did the create action.
  */
+// eslint-disable-next-line no-unused-vars
 Hooks.on("preCreateActor", async (actor, creationData, options, userId) => {
   // On brand new actors, creationData only has properties: `name`, and `type`.
   // If creationData has `system` then the new actor is either duplicated or imported,

module/roll/roll.js

@@ -42,6 +42,7 @@ export class DGRoll extends Roll {
    *                                        true, or the Object of prepared chatData otherwise.
    */
   async toMessage(messageData = {}, { rollMode, create = true } = {}) {
+    // eslint-disable-next-line no-param-reassign
     messageData.speaker = ChatMessage.getSpeaker({ actor: this.actor });
     return super.toMessage(messageData, {
       rollMode: this.options.rollMode || rollMode,
@@ -71,6 +72,7 @@ export class DGPercentileRoll extends DGRoll {
    * @param {DeltaGreenItem}  [options.item]     Optional - The item from which the roll originates.
    * @param {DeltaGreenItem}  [options.specialTrainingName] Optional - Special training rolls have names that are different from the roll key.
    */
+  // eslint-disable-next-line default-param-last, no-unused-vars
   constructor(formula = "1D100", data = {}, options) {
     super("1D100", {}, options);