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

Fix issues #3

Merged
merged 2 commits into from
Jul 31, 2023
Merged

Fix issues #3

merged 2 commits into from
Jul 31, 2023

Conversation

tsegismont
Copy link
Contributor

Fix remaining issues

Follows-up on #1 and #2

Signed-off-by: Thomas Segismont <tsegismont@gmail.com>
Signed-off-by: Thomas Segismont <tsegismont@gmail.com>
@tsegismont tsegismont requested a review from vietj July 31, 2023 09:34
@tsegismont
Copy link
Contributor Author

@vietj it seems I cannot push to main branch on this repo so I had to create this PR

@tsegismont tsegismont merged commit 017bf1d into eclipse-vertx:main Jul 31, 2023
5 checks passed
@tsegismont
Copy link
Contributor Author

@vietj main branch has been built and artifacts uploaded 🚀

@tsegismont tsegismont deleted the fix_issues branch July 31, 2023 09:51
@vietj
Copy link
Member

vietj commented Jul 31, 2023

so we can merge vertx-core PR now, is that right ?

@tsegismont
Copy link
Contributor Author

so we can merge vertx-core PR now, is that right ?

Yes

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

2 participants