Skip to content

Support passing diff options to React components#416

Open
gsabran wants to merge 1 commit intopierrecomputer:mainfrom
gsabran:whitespace-changes
Open

Support passing diff options to React components#416
gsabran wants to merge 1 commit intopierrecomputer:mainfrom
gsabran:whitespace-changes

Conversation

@gsabran
Copy link

@gsabran gsabran commented Mar 24, 2026

Description

Support passing diff options to React components

Motivation & Context

I wanted a way to ignore whitespace only changes

Type of changes

  • Bug fix (non-breaking change which fixes an issue)
  • Refactoring (non-breaking change)
  • New feature (non-breaking change which adds functionality). You must have
    first discussed with the dev team and they should be aware that this PR is
    being opened
  • Breaking change (fix or feature that would change existing functionality).
    You must have first discussed with the dev team and they should be aware
    that this PR is being opened
  • Documentation update

Checklist

  • I have read the
    contributing guidelines
  • My code follows the code style of the project (bun run lint)
  • My code is formatted properly (bun run format)
  • I have updated the documentation accordingly (if applicable)
  • I have added tests to cover my changes (if applicable)
  • All new and existing tests pass (bun run diffs:test)

How was AI used in generating this PR

I directed Claude to make the changes, reviewed and tested them carefully manually

@vercel
Copy link

vercel bot commented Mar 24, 2026

@gsabran is attempting to deploy a commit to the Pierre Computer Company Team on Vercel.

A member of the Team first needs to authorize it.

* - `ignoreWhitespace`: treat lines differing only in whitespace as unchanged
* - `stripTrailingCr`: strip `\r` before diffing (useful for UNIX vs Windows)
*/
diffOptions?: DiffComputeOptions;
Copy link
Author

Choose a reason for hiding this comment

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

maybe not the clearest property name? diffOptions within BaseDiffOptions 🤷‍♂️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant