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

chore: add devcontainer with java17 #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mikaello
Copy link

Thanks for a very nice utility (+1 to #1)!

Since the heroku-site is down, it may be nice to give users a devcontainers file with correct java version, that way they are only a click away from having a dev environment with Java 17 installed and ready to install with Maven and run the jar-files.

image

@Freed-Wu
Copy link

Freed-Wu commented Feb 28, 2024

I think package it to Linux package manager is more convenient.

Related issues:

@Freed-Wu
Copy link

@ris58h Can you publish a version tag? It can help packager to package this program 😄

@mikaello
Copy link
Author

I think package it to Linux package manager is more convenient.

I don't fully agree. By solely supporting a Linux package manager, I would need to install the package on my computer to run the software. However, with the approach proposed in this merge request, users can perform all tasks in the browser without the necessity of installing anything on their computers.

I believe giving the users the option to choose for themselves would be best. :-)

@Freed-Wu
Copy link

I would need to install the package on my computer to run the software

Oh, I see. You only need exported APK, not this program. In my situation, I must try many times to find a propitiate layout: swap two keys, try for a long time, feel not good and reexport a new APK with swapping other keys and try again. So I hope it can installed in my machine for convenience.

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