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

[jarun#753] temporarily block flake-admin-v2 & actualize install instructions #754

Merged
merged 1 commit into from
Jul 20, 2024

Conversation

LeXofLeviafan
Copy link
Collaborator

@LeXofLeviafan LeXofLeviafan commented Jul 20, 2024

except Exception:
app.logger.warning('failed to init locale')
except Exception as e:
app.logger.warning(f'failed to init locale ({e})')
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This includes the error message into the warning, making it easier to tell what happened

$ pip3 install ".[server]"
$ # server with reverse_proxy
$ pip3 install ".[server,reverse_proxy]"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

[reverse_proxy] is no longer present in setup.py

@@ -20,33 +20,28 @@ So be sure to have `python3`, `python3-pip` , `python3-dev`, `libffi-dev` packag
#### Dependencies

```
$ python3 -m pip install --user --upgrade pip
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Depending on OS, pip install --user is no longer allowed (unless you pass --break-system-packages as well)

@@ -20,33 +20,28 @@ So be sure to have `python3`, `python3-pip` , `python3-dev`, `libffi-dev` packag
#### Dependencies

```
$ python3 -m pip install --user --upgrade pip
$ python3 -m pip install --user virtualenv
$ python3 -m virtualenv env
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Installing virtualenv is no longer necessary: the venv module is included in basic Python installation (and accomplishes the same result)

@jarun jarun merged commit 615c0be into jarun:master Jul 20, 2024
1 check passed
@jarun
Copy link
Owner

jarun commented Jul 20, 2024

Thank you!

@LeXofLeviafan LeXofLeviafan deleted the flask-babel-v2 branch August 24, 2024 17:40
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