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

Compile project on Java 11, build on 11, 17, 21 #70

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

Conversation

szatyinadam
Copy link

@szatyinadam szatyinadam commented Nov 28, 2023

Hi @bryancross ,

I've just found your library and it eases the work for my team. I would like to do some contributions if You accept.
First of all, I think it would be good if JPostman could be used with lower Java versions than 18.
I made some small changes to enable the compilation of JPostman with Java 11 (Java 8 compatibility can be achieved if we use something other than java.net.http.HttpClient).

I've modified the build file to build the application with JDK 11, 17, 21, just to make sure it's working with these.

You can see the output of the job in my forked repository here or here is the snapshot build, but I do not have the Postman API key. That's why some of the tests fail, but it should work on your repository.

After this PR, I would like to work on the Maven Central publishing to be able to access JPostman as Maven dependency.

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