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.

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

asearch #61

Closed
wchodor opened this issue Nov 7, 2015 · 8 comments
Closed

asearch #61

wchodor opened this issue Nov 7, 2015 · 8 comments
Assignees
@wchodor
Labels
archivist text review

Comments

@wchodor
Copy link
Owner

wchodor commented Nov 7, 2015

No description provided.

@wchodor wchodor self-assigned this Nov 7, 2015
@wchodor
Copy link
Owner Author

wchodor commented Nov 30, 2015

Póki co mam pewne wątpliwości co do asearch. Poczekam na odpowiedź Przemka.

@wchodor
Copy link
Owner Author

wchodor commented Nov 30, 2015

@MarcinKosinski
Oto ta wątpliwość:
pbiecek/archivist#149

@wchodor
Copy link
Owner Author

wchodor commented Nov 30, 2015

ustalić poprzez aoptions parametr type

@wchodor
Copy link
Owner Author

wchodor commented Dec 1, 2015

Zrobione. Dużo zmian. Oto link do commita:
https://github.com/wchodor/archivist/commit/b7bef3279b8e8ba6fb0a85c43f1e7015b2f41811

@MarcinKosinski
Copy link
Collaborator

Juzpacze

@MarcinKosinski
Copy link
Collaborator

Swietnie! Widze loadFromRepo i SetRepo doczekalo sie swoich krotszych wersji, podobnie jak multiSearchInRepo :)

Czy udaloby sie to powtorzyc dla searchInRepo, cloneFromRepo, summaryRepo, showRepo i zipRepo? W jakims nowym zadanku :)?

@wchodor wchodor added Ready and removed in progress labels Dec 1, 2015
@wchodor
Copy link
Owner Author

wchodor commented Dec 1, 2015

Myślę, że tak. Na razie zrobiłem to na potrzeby asearch. Ale oczywiście reszta też musi być uaktualniona!

@wchodor
Copy link
Owner Author

wchodor commented Dec 1, 2015

Oto link do commita z poprawkami:
https://github.com/wchodor/archivist/commit/2a812bab8221a92b74e7c4218982ba431d1511f4

@wchodor wchodor closed this as completed Dec 1, 2015
@wchodor wchodor removed the Ready label Dec 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wchodor wchodor

Labels
archivist text review
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@MarcinKosinski @wchodor