Refactor before import warning#11773
Open
philderbeast wants to merge 1 commit into
Open
Conversation
7fc0f5e to
83257e1
Compare
6 tasks
ulysses4ever
approved these changes
May 3, 2026
Collaborator
ulysses4ever
left a comment
There was a problem hiding this comment.
Thanks a lot for splitting it --- it helped reviewing a lot! A couple nitpicks inline.
83257e1 to
0874d33
Compare
7310351 to
069d489
Compare
Collaborator
|
One little nitpick: I can't quite parse the PR title "Refactor/before import warning". Could you turn into something more readable? |
Collaborator
Author
|
Sorry about that @ulysses4ever. That was the branch name. I've removed the needless "/" separator. |
- Move fetchImportConfig to Import module - Move fetchImportParse to Import module - Nest fetchImportConfig within fetchImportParse - Move cyclicalImportMsg to cabal-install - Move untrimmedUriImportMsg to cabal-install - Move docProjectConfigFiles to cabal-install - Add compareLexically & compareSegmentally - Update hops test expectation for lexical sort - Remove +legacy-comparison - Satisfy fourmolu - Add change log entry - s/lexically/lexicographically/ - s/fetchImportParse/fetchImport/
069d489 to
de5211a
Compare
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.
Split from #10933, the refactoring and the sorting changes without the duplicate import by alternate paths detection, as requested by @ulysses4ever.
cabal-install/src/Distribution/Client/ProjectConfig/Import.hsand move some stuff there fromcabal-install-solver/src/Distribution/Solver/Types/ProjectConfigPath.hsthat was only ever used incabal-install.For messages, I added some doctests and ended up with two ways of comparing
ProjectConfigPath.I'll squash commits before applying the merge label if this pull request is approved.
Template Α: This PR modifies behaviour or interface
Include the following checklist in your PR:
significance: significantin the changelog file.