Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix1 #569

Closed
wants to merge 3 commits into from
Closed

Fix1 #569

wants to merge 3 commits into from

Conversation

ThePrimeagen
Copy link
Owner

@sug44 here you go

@ThePrimeagen
Copy link
Owner Author

a couple of breaking tests... looking

@ThePrimeagen
Copy link
Owner Author

@sug44 removing exact path has caused bugs

@sug44
Copy link

sug44 commented Apr 9, 2024

the tests were breaking on the parent commit as well, so i didnt try to fix any of them

@ThePrimeagen
Copy link
Owner Author

yeah, but the removal has caused somethings

there is also the usage of realpath that is causing breaking

@ThePrimeagen
Copy link
Owner Author

also... i realized... i cannot have this change

i am going to have to think of something different.

this change breaks one of the core concepts of harpoon. i can have 2 repos, both with the same key() in settings (think workspaces) and it will return the same results. i must have relative paths

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