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

Added __init__.py and util.py to endpoints #32

Merged
merged 1 commit into from
Jul 15, 2024

Conversation

brngylni
Copy link
Contributor

No description provided.

Copy link
Member

@gridhead gridhead left a comment

Choose a reason for hiding this comment

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

After making the suggested changes, please squash your commits and then it should be ready for merge.

Comment on lines 6 to 8



Copy link
Member

Choose a reason for hiding this comment

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

Leave no more than two lines here after every function definition.

@@ -0,0 +1,35 @@
from flask import Response, Request, request, abort
from functools import wraps

Copy link
Member

Choose a reason for hiding this comment

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

Leave no more than two lines here after every function definition.

Copy link
Member

@gridhead gridhead left a comment

Choose a reason for hiding this comment

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

I am not dragging this any longer than it has to be. Other required changes can be made to the codebase after it is merged.

Merging now.

@gridhead gridhead merged commit 524ee0b into fedora-infra:main Jul 15, 2024
3 of 10 checks passed
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