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

app_factory: add init functions entry points #88

Merged
merged 5 commits into from
Jun 26, 2023

Conversation

jrcastro2
Copy link
Contributor

@utnapischtim utnapischtim marked this pull request as ready for review June 21, 2023 21:08
* this addition makes gives the opportunity to add finalize entry points
  to the create_api factory
Copy link
Member

@slint slint left a comment

Choose a reason for hiding this comment

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

LGTM! I guess once we release invenio-base you can bump also the minimum version.

CHANGES.rst Outdated
@@ -8,6 +8,12 @@
Changes
=======

Version 1.4.0 (released 2023-06-23)

- Adds finalize_app and api_finalize_app entry points to the creation of the
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- Adds finalize_app and api_finalize_app entry points to the creation of the
- Adds ``invenio_base.finalize_app`` and ``invenio_base.api_finalize_app`` entry points to the creation of the

nit: for easier grepping in the future.

* every package which is used as an import statement should be added to
  the install requires explicitly

* removed the upper limit from pytest-black. it is not necessary.
@slint slint force-pushed the add-init-functions branch from 5d99a1b to e7ad584 Compare June 26, 2023 12:28
@slint slint force-pushed the add-init-functions branch from e7ad584 to a98c3e0 Compare June 26, 2023 12:29
@slint slint merged commit a98c3e0 into inveniosoftware:master Jun 26, 2023
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.

before_(app_)first_request deprecation and removal in Flask v2.3
3 participants