-
-
Notifications
You must be signed in to change notification settings - Fork 28
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
Restructure Repository #66
Conversation
Move proxytypes in subdirectory
Update `pyproject.toml` and add `pdm.lock`
Move tests into `test/` directory and split them into multiple files
Fix flake8 errors
Add `black` to dev dependencies and fix CI
Correctly configure package build
Looks good to me, thanks. Seems like some of the tests got moved out of the main file, but I don't see those tests removed from the main file? |
Remove duplicate tests
I split the test classes in I can revert it, of course. Just found that to look cleaner. EDIT: In the commit 1eba891, I removed some tests I forgot to delete. Before and after the change there are 107 pytests, so all should be there. |
Yep, that's what I meant, looks good now. |
Make it possible to `from jsonref import LazyProxy` again Specify `# noqa` comment more
Awesome. Thanks so much! |
Sure :) Would not like to have a package conflict somewhere down the road when two packages want to install the same filenames. Thank you for merging it. |
This closes #64.
tests/
andjsonref
directories instead of having all files in rootblack
andflake8
as dev dependencies (next topytest
)pyproject.toml
use[project.optional-dependencies]
notation instead of[tool.pdm.dev-dependencies]
because then one can simply usepip install ".[dev]"
to install optional dependenciesThis is transparent to users. All import paths are the same.