StartDown/AuthMicroservice
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

app/routers/demo.py: minor changes #2

Closed
Aklivtsov wants to merge 1 commits from dev into main
pull from: dev
merge into: StartDown:main
Conversation 0 Commits 1 Files Changed 1 +16 -16
Aklivtsov commented 2025-12-01 13:36:04 +03:00
Owner
Copy Link

Added early exits ('cause optimization), removed variable for HTTPExeption ('cause yeah, DRY is cool, but we don't need to apply it to every line of code), fixed typings (I've added types, that function returns). Overall, your function stays the same.

Added early exits ('cause optimization), removed variable for HTTPExeption ('cause yeah, DRY is cool, but we don't need to apply it to every line of code), fixed typings (I've added types, that function returns). Overall, your function stays the same.
Aklivtsov added 1 commit 2025-12-01 13:36:05 +03:00
app/routers/demo.py: minor changes fd55ce7987
Aklivtsov requested review from roma 2025-12-01 13:36:36 +03:00
roma closed this pull request 2025-12-01 13:54:59 +03:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
roma
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
Aklivtsov LeterZP roma
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: StartDown/AuthMicroservice#2
Delete Branch "dev"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?