fix: rename hook use-environment-links#2410
Merged
RemiBonnet merged 1 commit intostagingfrom Feb 23, 2026
Merged
Conversation
Co-authored-by: Rémi Bonnet <bonnet.rem@gmail.com>
|
Cursor Agent can help with this pull request. Just |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## staging #2410 +/- ##
===========================================
+ Coverage 48.39% 48.72% +0.33%
===========================================
Files 939 696 -243
Lines 19288 15471 -3817
Branches 5753 4651 -1102
===========================================
- Hits 9335 7539 -1796
+ Misses 8181 6513 -1668
+ Partials 1772 1419 -353
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
rmnbrd
approved these changes
Feb 23, 2026
Member
Author
|
🎉 This PR is included in version 1.290.2 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Issue: N/A
This PR renames the
use-env-linkshook and all its references touse-environment-linksfor improved clarity and consistency.Changes:
libs/domains/services/feature/src/lib/hooks/use-env-links/use-env-links.tstouse-environment-links.ts.libs/domains/services/feature/src/lib/service-links-popover/service-links-popover.tsx.jest.mockandjest.requireActualpaths inlibs/domains/services/feature/src/lib/service-links-popover/service-links-popover.spec.tsx.libs/domains/services/feature/src/index.ts.Reasons / problems solved: Enhances code readability and adheres to more descriptive naming conventions.
Reviewer Instructions:
use-env-linksno longer exists in the codebase.use-environment-links.Screenshots / Recordings
Testing
yarn testoryarn test -u(if you need to regenerate snapshots) - AI could not run due to missingnode_modulesyarn formatyarn lintPR Checklist
.cursor/rules)feat(service): add new Terraform service) - required for semantic-releaseNote
Low Risk
Pure rename and reference updates; risk is limited to broken imports/exports if any downstream usage was missed.
Overview
Renames the services feature hook
use-env-linkstouse-environment-linksand updates the public barrel export accordingly.Updates
ServiceLinksPopoverand its Jest tests to import/mock the hook from the new path, with no functional behavior changes beyond the rename.Written by Cursor Bugbot for commit 15720c6. This will update automatically on new commits. Configure here.