-
Notifications
You must be signed in to change notification settings - Fork 4
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
Feature-authorized-scopes-updater #560
Conversation
🧙 Sourcery has finished reviewing your pull request! Tips
|
🔍 Existing Issues For ReviewYour pull request is modifying functions with the following pre-existing issues: 📄 File: ebl/fragmentarium/web/bootstrap.py
📄 File: ebl/fragmentarium/web/fragments.py (Click to Expand)
Did you find this useful? React with a 👍 or 👎 |
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.
We've reviewed this pull request using the Sourcery rules engine. If you would also like our AI-powered code review then let us know.
resp.status = falcon.HTTP_200 | ||
resp.media = create_response_dto(updated_fragment, user, has_photo) | ||
except ValueError as error: | ||
raise DataError(error) |
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.
suggestion (code-quality): Explicitly raise from a previous error (raise-from-previous-error
)
raise DataError(error) | |
raise DataError(error) from error |
* adapt on_get to return provenance parents in square brackets * reformatting * Fixed test for provenance
* Remove user scopes from latest addition aggregation Only include "open for all" fragments * add test_query_latest_skips_restricted_fragments
* update ScopeField * update function call * update test data * update match_user_scopes
* install ruff * reformat files * fix mocking calls to user profile * suppress "Unable to concatenate tuple [60]" error * fix formatting error * refactor * update tasks * use ruff for linting and formatting * remove old config file * bugfix * add ruff * replace tuple() with () * remove C408 from ignored rules * replace `tuple()` with literal `()` and reformat
…m/ElectronicBabylonianLiterature/ebl-api into feature-authorized-scopes-updater
…m/ElectronicBabylonianLiterature/ebl-api into feature-authorized-scopes-updater
Code Climate has analyzed commit 9b4c99b and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 92.3% (75% is the threshold). This pull request will bring the total coverage in the repository to 91.4% (0.0% change). View more on Code Climate. |
Added updater of authorized_scopes field in Fragment based on Genre updater