-
Notifications
You must be signed in to change notification settings - Fork 104
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
Consistent lockfile-relative vs absolute sources #230
Consistent lockfile-relative vs absolute sources #230
Conversation
✅ Deploy Preview for conda-lock ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
On hold waiting for PR #204, see #204 (comment) |
@riccardoporreca @maresb I think #204 would be merged - anything I could help with here? |
Looks like there are merge conflicts to be resolved |
@bstadlbauer, you could rebase @riccardoporreca's branch on |
@bstadlbauer, @maresb, if this can wait till the weekend, I will be happy to take care of the rebasing |
@riccardoporreca Could we add a test that checks the new behavior? Just to make sure there is no regression in future versions |
@riccardoporreca Is it alright to add the missing tests for you? Ran into this issue recently and would love to see this PR merged. |
@maresb, @bstadlbauer, I was too optimistic with my time before the festivities in December :), thanks @srilman for the reminder. I will manage to rebase and at least draft some tests in the next few days |
See #328 for the rebased/revamped PR |
Resolve #229