pierrejeambrun opened a new pull request, #70520:
URL: https://github.com/apache/airflow/pull/70520

   Backport-ish of #70402 to `v3-3-test`.
   
   Not a clean cherry-pick: #70402 bumped `eslint-plugin-unicorn` 71.1.0 → 
72.0.0 on `main`, but `v3-3-test` was pinned at `^64.0.0`, so this is a larger 
jump. Rather than hand-merge the diverged `pnpm-lock.yaml`, the lock was 
regenerated for just this dependency (`pnpm install`).
   
   The newer unicorn rules surface some findings, fixed here:
   - `eslint --fix` auto-resolved the fixable ones (numeric separators, etc.).
   - 2 `prefer-ternary` (source: `usePluginTabs`, `TaskInstance`).
   - 2 `consistent-function-scoping` on test-local helper components 
(`MarkdownModal.test`, `CreateAssetEventModal.test`) — kept in place with a 
documented `eslint-disable` (one must stay inside a `vi.mock` factory due to 
vitest hoisting).
   
   `eslint --quiet` (CI mode) passes with 0 errors.
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Code (Opus 4.8)
   
   Generated-by: Claude Code (Opus 4.8) following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to