Skip to content

chore(generator-cli): remove .fern/replay.yml from REPLAY_FERNIGNORE_ENTRIES#14812

Open
tstanmay13 wants to merge 1 commit intomainfrom
devin/1775707950-remove-replay-yml-from-fernignore
Open

chore(generator-cli): remove .fern/replay.yml from REPLAY_FERNIGNORE_ENTRIES#14812
tstanmay13 wants to merge 1 commit intomainfrom
devin/1775707950-remove-replay-yml-from-fernignore

Conversation

@tstanmay13
Copy link
Copy Markdown
Contributor

Description

Removes .fern/replay.yml from the list of files that fern replay init adds to .fernignore.

Changes Made

  • Removed .fern/replay.yml from the REPLAY_FERNIGNORE_ENTRIES constant in fernignore.ts, reducing it from 3 entries to 2 (.fern/replay.lock and .gitattributes)
  • Updated the corresponding test assertions in replay-pure.test.ts

Human Review Checklist

  • Confirm .fern/replay.yml no longer needs to be preserved across generations (i.e., it's fine for fern generate to overwrite it)
  • Verify no downstream consumers (e.g., fiddle server-side) explicitly depend on .fern/replay.yml being in the fernignore entries list

Testing

  • Unit tests updated
  • Lint (biome check) passes

Link to Devin session: https://app.devin.ai/sessions/48f13de8b82c444db51e0a931c798a3d
Requested by: @tstanmay13

…ENTRIES

Co-Authored-By: tanmay.singh <tstanmay13@gmail.com>
Copy link
Copy Markdown

@claude claude bot left a comment

Choose a reason for hiding this comment

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

Tip: disable this comment in your organization's Code Review settings.

@devin-ai-integration
Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants