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

Run tests on a different port #869

Merged
merged 2 commits into from
Feb 13, 2019

Conversation

stuartwdouglas
Copy link
Member

I have gone with 8081, but I am not sure if we should go something really random instead

@gsmet
Copy link
Member

gsmet commented Feb 13, 2019

@stuartwdouglas not merging just yet as I don't know if you want to have the random port discussion here. Feel free to merge if not.

@Sanne
Copy link
Member

Sanne commented Feb 13, 2019

@gsmet this was discussed at the meeting yesterday? Recap: Need just to separate the testing port for now. The random port thing is a next-step enhancement, cant wait a bit more.

@gsmet
Copy link
Member

gsmet commented Feb 13, 2019

@Sanne sure but see the issue description :).

I approved it, just letting Stuart deciding when he wants to merge it.

@stuartwdouglas stuartwdouglas merged commit bab55ee into quarkusio:master Feb 13, 2019
@stuartwdouglas
Copy link
Member Author

I will just merge it for now, if people want to do something different then we can revisit later

@stuartwdouglas stuartwdouglas added this to the 0.9.0 milestone Feb 14, 2019
@Sanne Sanne deleted the devmode branch February 14, 2019 13:11
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.

None yet

3 participants