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

Introduce Process.getAllProcess #80

Merged
merged 1 commit into from
Mar 8, 2021
Merged

Conversation

gastaldi
Copy link
Contributor

@gastaldi gastaldi commented Mar 8, 2021

Fixes #79

@gastaldi
Copy link
Contributor Author

gastaldi commented Mar 8, 2021

@geoand see if this change fits your use case

@geoand
Copy link
Contributor

geoand commented Mar 8, 2021

Yeah that would likely work

Copy link
Collaborator

@dmlloyd dmlloyd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this idea is okay, just a couple of comments.

@gastaldi gastaldi force-pushed the all_process branch 4 times, most recently from 5f20b95 to 1d5b01f Compare March 8, 2021 16:27
@gastaldi gastaldi force-pushed the all_process branch 5 times, most recently from 781abf5 to f4ae3c6 Compare March 8, 2021 16:53
@gastaldi
Copy link
Contributor Author

gastaldi commented Mar 8, 2021

Finally all green!

@gastaldi gastaldi requested a review from dmlloyd March 8, 2021 17:05
@gastaldi gastaldi force-pushed the all_process branch 2 times, most recently from 902db22 to c7304d0 Compare March 8, 2021 17:13
Fixes smallrye#79

Co-authored-by: David M. Lloyd <[email protected]>
@dmlloyd dmlloyd merged commit 26661c1 into smallrye:master Mar 8, 2021
@gastaldi gastaldi deleted the all_process branch March 8, 2021 17:23
@radcortez radcortez added this to the 1.5.1 milestone Mar 17, 2021
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.

Add method to retrieve all running processes
4 participants