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

build: remove Dockerfile, add codespace config, and bump min python #199

Merged
merged 12 commits into from
Dec 19, 2023

Conversation

aryarm
Copy link
Member

@aryarm aryarm commented Dec 16, 2023

This PR removes our now-defunct Dockerfile. Instead, we recommend that you use the TRTools image maintained by the Bioconda team on the biocontainers registry. The docs have been updated to reflect this.

To offset the removal of the Dockerfile, this PR adds an alternative way to work with TRTools in the cloud: GitHub codepsaces! This can be helpful when you want to quickly play around with our code but don't want to go through all of the setup steps. Just click on the "Open in Github Codespaces" badge on the README and voila - you'll be presented with a VS Code environment automagically set up with TRTools and our development dependencies. This environment will run with (usually free-for-academic-use) credits on Github's codespace VMs.

Finally, this PR also specifies a strategy for our support for various python versions. From now on, we will only formally support active versions of python that are not end-of-life (3.8+). However, at least for now, we think that TRTools will still run properly for versions 3.6 and 3.7 - although support for those versions is not guaranteed and may break at any time.

@aryarm aryarm requested a review from aarushi03 December 16, 2023 06:21
@aryarm
Copy link
Member Author

aryarm commented Dec 16, 2023

@aarushi03, can I request your review on this PR, since you've worked with GitHub codespaces before? Can you try out the new badge (checked out to the build/Dockerfile branch) and let me know if it works for you?

@aarushi03
Copy link
Contributor

@aarushi03, can I request your review on this PR, since you've worked with GitHub codespaces before? Can you try out the new badge (checked out to the build/Dockerfile branch) and let me know if it works for you?

Certainly. Reviewing it now.

.devcontainer/environment.yml Outdated Show resolved Hide resolved
README.rst Show resolved Hide resolved
README.rst Outdated Show resolved Hide resolved
@aryarm aryarm changed the title build: remove Dockerfile and add codespace config build: remove Dockerfile, add codespace config, and bump min python Dec 16, 2023
@aryarm aryarm merged commit 1527df6 into master Dec 19, 2023
5 checks passed
@aryarm aryarm deleted the build/Dockerfile branch December 19, 2023 20:32
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