-
Notifications
You must be signed in to change notification settings - Fork 0
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(#9): home endpoints #10
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@h1alexbel i think that it would be better to move home.rs
and rs_err.rs
to different folders and include them from modules
clippy.toml
Outdated
disallowed-names = ["not-jeff"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@h1alexbel this one was like an example
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@l3r8yJ fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@h1alexbel looks good to me!
@rultor merge |
@h1alexbel OK, I'll try to merge now. You can check the progress of the merge here |
@h1alexbel Oops, I failed. You can see the full log here (spent 10min)
|
blocked by #11 |
@l3r8yJ can we try to install required |
@rultor merge |
@h1alexbel OK, I'll try to merge now. You can check the progress of the merge here |
@rultor stop |
@rultor merge |
@h1alexbel OK, I'll try to stop current task. |
@h1alexbel Sorry, I failed to stop the previous command, however it has the following result: Oops, I failed. You can see the full log here (spent 2hr)
|
@h1alexbel OK, I'll try to merge now. You can check the progress of the merge here |
@h1alexbel Oops, I failed. You can see the full log here (spent 8min)
|
@rultor merge |
1 similar comment
@rultor merge |
@h1alexbel looks like we need to debug rultor... |
@l3r8yJ it's in outage for now |
@h1alexbel you can close this pr, remove remote branch and reopen pr, maybe it will help |
@l3r8yJ this repository is not connected with this outage, we can only wait for now |
@h1alexbel whatsupp here? |
@l3r8yJ |
@h1alexbel sure, wont wait |
@l3r8yJ take a look, please
ref #9
PR-Codex overview
The focus of this PR is to add new dependencies, implement routes for home endpoints, and create error handling structures.
Detailed summary
serde_json
dependencyaxum
andtokio
dependencies