Skip to content

disable unicorn/prefer-single-call for spec.ts and test.ts files#236

Merged
jpolavar merged 3 commits intomainfrom
unicorn/prefer-single-call-disable-tests
Aug 11, 2025
Merged

disable unicorn/prefer-single-call for spec.ts and test.ts files#236
jpolavar merged 3 commits intomainfrom
unicorn/prefer-single-call-disable-tests

Conversation

@jpolavar
Copy link
Contributor

Closes #230

@jpolavar jpolavar self-assigned this Jul 15, 2025
Copilot AI review requested due to automatic review settings July 15, 2025 20:33
@jpolavar jpolavar added the PATCH label Jul 15, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR disables the unicorn/prefer-single-call ESLint rule for spec.ts and test.ts files and bumps the package version. The change addresses issue #230 by allowing more flexible coding patterns in test files where multiple method calls may be preferred for readability.

  • Disables unicorn/prefer-single-call rule for test files
  • Increments package version from 11.2.7 to 11.2.8

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
package.json Version bump to reflect the configuration change
index.mjs Adds rule override to disable unicorn/prefer-single-call in test configurations

@jpolavar jpolavar requested review from le-cong and ramaghanta and removed request for le-cong and ramaghanta July 15, 2025 20:37
Copy link
Contributor

@ramaghanta ramaghanta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lftm

@github-actions
Copy link

github-actions bot commented Aug 8, 2025

Beta Published - Install Command: npm install @checkdigit/eslint-config@11.3.1-PR.236-c7e7

@jpolavar jpolavar requested a review from ramaghanta August 8, 2025 00:04
Copy link
Contributor

@ramaghanta ramaghanta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lftm

@jpolavar jpolavar merged commit 5d44cf4 into main Aug 11, 2025
8 checks passed
@jpolavar jpolavar deleted the unicorn/prefer-single-call-disable-tests branch August 11, 2025 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

disable unicorn/prefer-single-call for spec.ts and test.ts files

3 participants