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

fix: http error #94

Merged
merged 11 commits into from
Jan 2, 2025
Merged

fix: http error #94

merged 11 commits into from
Jan 2, 2025

Conversation

joydeep049
Copy link
Contributor

Closes #92

Fairly simple fix. Should deal with the problem.
Tested it locally and it works.

cc @cyclotruc

Co-authored-by: Filip Christiansen <[email protected]>
@filipchristiansen
Copy link
Collaborator

Could you add a test or two?

src/gitingest/parse_query.py Outdated Show resolved Hide resolved
src/gitingest/parse_query.py Outdated Show resolved Hide resolved
cyclotruc and others added 2 commits January 2, 2025 00:43
Co-authored-by: Filip Christiansen <[email protected]>
Co-authored-by: Filip Christiansen <[email protected]>
src/gitingest/tests/test_parse_query.py Outdated Show resolved Hide resolved
@cyclotruc cyclotruc added the changes requested A review requested changes before merging label Jan 2, 2025
@cyclotruc cyclotruc merged commit 8d58d53 into cyclotruc:main Jan 2, 2025
8 checks passed
@cyclotruc
Copy link
Owner

Merged! Thank you @joydeep049

@joydeep049
Copy link
Contributor Author

Thank you @cyclotruc
Actually I wanted to discuss something, could you please check discord at your convenience

Best

@filipchristiansen filipchristiansen removed the changes requested A review requested changes before merging label Jan 10, 2025
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.

Git Ingest request fails on web if the input repo url starts with "http://" instead of "https://"
4 participants