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.

Sign up for GitHub

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

Jump to bottom

Move to single pyproject.toml build #28

Closed
kwinkunks opened this issue Jan 2, 2023 · 0 comments · Fixed by #42
Closed

Move to single pyproject.toml build #28

kwinkunks opened this issue Jan 2, 2023 · 0 comments · Fixed by #42
Assignees
@kwinkunks
Labels
enhancement New feature or request

Comments

@kwinkunks
Copy link
Member

Some things might be easier:

  • Currently have requirements in 4 different files (setup.py, requirements.txt, docs_requirements.txt and test_requirements.txt) .These could be consolidated as installation options in pyproject.toml (for example; lots of ways to do it).
  • Would not need the requirements files, setup.cfg or setup.py, and maybe can also get rid of conftest.py
@kwinkunks kwinkunks added the enhancement New feature or request label Jan 2, 2023
@kwinkunks kwinkunks self-assigned this Dec 19, 2023
kwinkunks added a commit that referenced this issue Dec 19, 2023
@kwinkunks
Modernize build system
b057f37 
Fixes #28. Also updating GitHub workflows, README/CONTRIBUTING, etc.
@kwinkunks kwinkunks mentioned this issue Dec 19, 2023
Merged
kwinkunks added a commit that referenced this issue Dec 19, 2023
@kwinkunks
Modernize build system
a5e7f13 
Fixes #28. Also updating GitHub workflows, README/CONTRIBUTING, etc.
kwinkunks added a commit that referenced this issue Dec 19, 2023
@kwinkunks
Modernize build system
491d54f 
Fixes #28. Also updating GitHub workflows, README/CONTRIBUTING, etc.
kwinkunks added a commit that referenced this issue Dec 20, 2023
@kwinkunks
Modernize build system
9567c0f 
Fixes #28. Also updating GitHub workflows, README/CONTRIBUTING, etc.
kwinkunks added a commit that referenced this issue Dec 20, 2023
@kwinkunks
Modernize build system
5f43ed5 
Fixes #28. Also updating GitHub workflows, README/CONTRIBUTING, etc.
kwinkunks added a commit that referenced this issue Dec 21, 2023
@kwinkunks
Modernize build system (#42)
7892da4 
* Modernize build system

Fixes #28. Also updating GitHub workflows, README/CONTRIBUTING, etc.

* Switch to using syrupy

pytest-snapshottest uses imp, which was deprecated in 3.11 and gone now.
They don't look like moving
syrusakbary/snapshottest#166

Conflicts:
	.github/workflows/python-build-test.yml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kwinkunks kwinkunks

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Modernize build system equinor/open_petro_elastic
1 participant
@kwinkunks