Skip to content

chore: add deprecated comment to FeaturesRepository#27611

Merged
sean-brydon merged 1 commit intomainfrom
devin/1770214014-deprecate-features-repository
Feb 4, 2026
Merged

chore: add deprecated comment to FeaturesRepository#27611
sean-brydon merged 1 commit intomainfrom
devin/1770214014-deprecate-features-repository

Conversation

@eunjae-lee
Copy link
Copy Markdown
Contributor

What does this PR do?

Adds a @deprecated JSDoc annotation to the FeaturesRepository class in packages/features/flags/features.repository.ts.

This will cause IDEs to show deprecation warnings when the class is used, signaling to developers that this class should not be used for new code.

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. N/A - documentation-only change.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works. N/A - no functional changes.

How should this be tested?

No testing required - this is a documentation-only change that adds a @deprecated JSDoc tag.

To verify:

  1. Open packages/features/flags/features.repository.ts in an IDE
  2. Confirm the @deprecated tag is present in the class JSDoc
  3. Import/use FeaturesRepository elsewhere and verify the IDE shows a deprecation warning

Requested by: @eunjae-lee
Link to Devin run: https://app.devin.ai/sessions/7628fd1fea3c433cba5bd8271a5f7911

Co-Authored-By: eunjae@cal.com <hey@eunjae.dev>
@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 that start with 'DevinAI' or '@devin'.
  • 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

@paragon-review
Copy link
Copy Markdown

paragon-review Bot commented Feb 4, 2026

Paragon Review Unavailable

Hi @eunjae-lee! You are not toggled active by your team. Reach out to your admin.

@sean-brydon sean-brydon marked this pull request as ready for review February 4, 2026 14:37
@graphite-app graphite-app Bot added core area: core, team members only consumer labels Feb 4, 2026
@graphite-app graphite-app Bot requested a review from a team February 4, 2026 14:37
@sean-brydon sean-brydon merged commit 8dde579 into main Feb 4, 2026
90 of 91 checks passed
@sean-brydon sean-brydon deleted the devin/1770214014-deprecate-features-repository branch February 4, 2026 14:37
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

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

Labels

consumer core area: core, team members only ready-for-e2e size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants