Skip to content

Conversation

nerdoc
Copy link
Contributor

@nerdoc nerdoc commented Jun 9, 2021

Python since v3.4 uses pathlib for paths preferrably: https://www.python.org/dev/peps/pep-0428/. Django, when using a new project, automatically imports Path in the settings:

# Build paths inside the project like this: BASE_DIR / 'subdir'.
BASE_DIR = Path(__file__).resolve().parent.parent

no need for that. And, BASE_DIR is declared automatically too, even if is no Python standard.

  • Tests are passing
  • Documentation not needed here.

Python since v3.4 uses pathlib for paths preferrably:  https://www.python.org/dev/peps/pep-0428/. Django, when using a new project, automatically imports `Path` in the settings:
```python
# Build paths inside the project like this: BASE_DIR / 'subdir'.
BASE_DIR = Path(__file__).resolve().parent.parent
```
no need for that. And, BASE_DIR is declared automatically too, even if is no Python standard.
@nerdoc nerdoc changed the title simplify code simplify example code Jun 9, 2021
nerdoc added 2 commits June 22, 2021 22:58
This is deprecated and will be removed in the next StimulusReflex versions
@jonathan-s
Copy link
Owner

Thank you for the reminder of application.consumer. Right now I'm planning to take over the control of what StimulusReflex is doing which you can see in #135. So this won't be relevant so I will close this PR. I appreciate you making it though!

@jonathan-s jonathan-s closed this Aug 1, 2021
@nerdoc nerdoc deleted the patch-2 branch August 8, 2021 10:19
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