From e401f43fe117ea7367a5b7bd19d11086b8a10bf9 Mon Sep 17 00:00:00 2001 From: Guillem Arias Date: Wed, 27 May 2026 10:47:30 +0200 Subject: [PATCH] fix(goals): Number.NaN over global NaN in goal_form_controller MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Biome's lint/style/useNumberNamespace rule. Same semantics — Number.NaN is the explicit namespace form post-ES2015. CI lint_js was failing on this line. --- app/javascript/controllers/goal_form_controller.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/javascript/controllers/goal_form_controller.js b/app/javascript/controllers/goal_form_controller.js index 74229eafd..251d50aae 100644 --- a/app/javascript/controllers/goal_form_controller.js +++ b/app/javascript/controllers/goal_form_controller.js @@ -81,7 +81,7 @@ export default class extends Controller { updateSuggested() { if (!this.hasSuggestedTarget) return; - const amount = this.hasAmountInputTarget ? Number.parseFloat(this.amountInputTarget.value) : NaN; + const amount = this.hasAmountInputTarget ? Number.parseFloat(this.amountInputTarget.value) : Number.NaN; const dateValue = this.hasDateInputTarget ? this.dateInputTarget.value : null; const checkedCount = this.linkedAccountCheckboxTargets.filter((cb) => cb.checked).length;