Skip to content

Altsearch #123

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 27 commits into
base: master
Choose a base branch
from
Open

Altsearch #123

wants to merge 27 commits into from

Conversation

@ASolovyev
Copy link

Todos

  • Documentation
  • Code Review, all remarks corrected
  • Unit (preferred) or internal crosspm-integration-tests added
  • Internal crosspm-integration-tests passed on dev-version crosspm
  • Internal cpmconfig passed on dev-version crosspm

Alexey Solovyev added 25 commits October 4, 2022 15:13
New altsearch branch. --altsearch=VALUE parameter to find packages fi…
f4f9970 
…rst in altsearch repo and then if not found in another repos
fix formatting
adb02fd
zaputalsya with formatting. fix
3f83f35
dig altsearch and altsearchbranch to chose funcs
ffc31a2
fix formatting
13b9d88
fix formatting vol. 2
de892aa
Merge branch 'master' into altsearch
49834db
Merge branch 'master' into altsearch
26e8395
Merge branch 'master' into altsearch
90141d3
fix merge problems
0fb067a
fix formatting issues
2585f02
fix parameters
138b2f1
get alt paths list
56358f0
refactor artifactory search to separate method
224c4d8
fix formatting
006c306
fix tests
3021a37
fix tests
e46c984
fix tests
e27aaac
yet another commit. added altpath vars
15585f8
it works
58eeb74
almost done. But ot yet
3025e43
fix formatting
0a289e2
seems like it works
e45fc72
fix unused variable self._packed_cache_p
89df3d2
fix comments and messages
8ed577c
tavx
tavx reviewed Nov 8, 2022
View reviewed changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@tavx tavx tavx left review comments

Reviewers whose approvals may not affect merge requirements

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ASolovyev @tavx