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

Miscellaneous improvements #17

Merged
merged 5 commits into from
Dec 6, 2024
Merged

Miscellaneous improvements #17

merged 5 commits into from
Dec 6, 2024

Conversation

aphedges
Copy link
Contributor

@aphedges aphedges commented Dec 6, 2024

This commit contains a variety of changes I've made over the past month while working on the library.

Generated automatically by the following commands:

```
pip install --upgrade --upgrade-strategy eager -r requirements-dev.txt
make lock
```
Now that I am using an Apple silicon laptop, Docker defaults to building
images for ARM. This image will always need to run on an x86_64 server,
so we should explicitly set the architecture for builds.
The separate `log/` directory was created before I started using
top-level `data/` directories in projects, and there is no need to keep
it separate. Moving it into `data/` makes the repository easier to work
with, which includes allowing cleaner `git clean` output.
I removed orchestration support for Apptainer and Singularity a while
ago, so there is no need to ignore their files anymore. If anyone wants
to use them, they can just store them in `data/`.
@aphedges aphedges self-assigned this Dec 6, 2024
@aphedges aphedges merged commit c6a354d into main Dec 6, 2024
2 checks passed
@aphedges aphedges deleted the misc-improvements-2024-12-05 branch December 6, 2024 00:36
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.

1 participant