Skip to content

Conversation

@grahamalama
Copy link
Contributor

Before, we had this project structure

src
├── __init__.py
├── app
├── jbi
├── static
├── templates

the differences between src/app and src/jbi weren't clear enough
to warrant separate packages, so in this PR we move most modules
into the root package, renamed to jbi/. We also change the structure
of the test files accordingly and update any other files that referred
to src/* files.

Before, we had this project structure
```
src
├── __init__.py
├── app
├── jbi
├── static
├── templates
```

the differenences between `src/app` and `src/jbi` weren't clear enough
to warrant separate pacakges, so in this commit we move most modules
into the root pacakge, renamed to `jbi/`. We also change the structure
of the test files accordingly and update any other files that refered
to src/*.
@grahamalama grahamalama requested a review from a team as a code owner August 2, 2022 13:45
@leplatrem leplatrem added the python Pull requests that update Python code label Aug 2, 2022
Copy link
Contributor

@bsieber-mozilla bsieber-mozilla left a comment

Choose a reason for hiding this comment

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

:shipit:

@grahamalama grahamalama merged commit 323a169 into main Aug 2, 2022
@grahamalama grahamalama deleted the slim-project-structure branch August 2, 2022 16:34
@leplatrem leplatrem mentioned this pull request Aug 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

python Pull requests that update Python code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants