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

feat: implement info/refs slice #12

Merged
merged 1 commit into from
Nov 22, 2021
Merged

feat: implement info/refs slice #12

merged 1 commit into from
Nov 22, 2021

Conversation

g4s8
Copy link
Member

@g4s8 g4s8 commented Nov 19, 2021

Add InfoRefs slice implementation which returns
server metadata and supported command that server can handle.
Based on this response git client decide what command to execute next.

Add GitResponseOutput to format ASCI text as git response
with length prefixes and part-end symbols.

Add main entry point to debug Artipie git server on localhost.

Update integration test GitITCase: setup bare repo on storage,
updated ls-remote test, added correct assertiion for ls-remote test.

Ticket: #1
Ticket: #11

@g4s8 g4s8 requested review from a team and olenagerasimova and removed request for a team November 19, 2021 14:52
@olenagerasimova
Copy link
Member

@g4s8 looks good to me, please, fix the build

Add `InfoRefs` slice implementation which returns
server metadata and supported command that server can handle.
Based on this response git client decide what command to execute next.

Add `GitResponseOutput` to format ASCI text as git response
with length prefixes and part-end symbols.

Add `main` entry point to debug Artipie git server on localhost.

Update integration test `GitITCase`: setup bare repo on storage,
updated `ls-remote` test, added correct assertiion for `ls-remote` test.

Ticket: #1
Ticket: #11
@g4s8
Copy link
Member Author

g4s8 commented Nov 22, 2021

@olenagerasimova thanks, fixed

@g4s8 g4s8 enabled auto-merge November 22, 2021 08:59
Copy link
Member

@olenagerasimova olenagerasimova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@g4s8 thanks

@g4s8 g4s8 merged commit 3f96556 into master Nov 22, 2021
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