Skip to content

Conversation

@kelly-sovacool
Copy link
Member

@kelly-sovacool kelly-sovacool commented Apr 28, 2025

Changes

there's no need for symlinks to use an absolute path when the target is in the same directory

image

Issues

NA

PR Checklist

(Strikethrough any points that are not applicable.)

  • This comment contains a description of changes with justifications, with any relevant issues linked.
  • Write unit tests for any new features, bug fixes, or other code changes.
  • [ ] Update docs if there are any API changes.
  • Update CHANGELOG.md with a short description of any user-facing changes and reference the PR number. Guidelines: https://keepachangelog.com/en/1.1.0/

@codecov
Copy link

codecov bot commented Apr 28, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.51%. Comparing base (0825a3e) to head (ade228b).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #58   +/-   ##
=======================================
  Coverage   72.51%   72.51%           
=======================================
  Files          21       21           
  Lines        1557     1557           
=======================================
  Hits         1129     1129           
  Misses        428      428           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@kelly-sovacool kelly-sovacool merged commit 4e265bb into main Apr 28, 2025
11 checks passed
@kelly-sovacool kelly-sovacool deleted the symlink-paths branch April 28, 2025 17:26
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.

2 participants