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 regression for Kotlin companion objects #229

Closed
wants to merge 3 commits into from
Closed

Fix regression for Kotlin companion objects #229

wants to merge 3 commits into from

Conversation

rgmz
Copy link
Contributor

@rgmz rgmz commented Dec 30, 2018

Simple workaround for vert-x3/vertx-lang-kotlin#93.

LMK if there's a better/more appropriate fix.

@rgmz
Copy link
Contributor Author

rgmz commented Dec 6, 2019

Hey @tsegismont,

I just wanted to bring some attention to this issue as it's still affecting Kotlin users (vert-x3/vertx-guide-for-java-devs#28).

@wusatosi
Copy link

@pmlopes @rgmz any updates?

@vietj
Copy link
Member

vietj commented Jan 6, 2020

can you sign the Eclipse agreement @rgmz ?

@rgmz
Copy link
Contributor Author

rgmz commented Jan 7, 2020

@vietj I have signed the agreement, I'm not sure why it's saying otherwise.
image

Do I need to amend the commit message for fdf83c8?

@okou19900722
Copy link
Contributor

okou19900722 commented Jan 7, 2020

all commit must signed-off with you email. now, only last commit signed-off

@rgmz
Copy link
Contributor Author

rgmz commented Jan 7, 2020

all commit must signed-off with you email. now, only last commit signed-off

@okou19900722 I'm not sure that I can do so without messing up the commit history.

I just tried to rebase and the result was:
image

@rgmz
Copy link
Contributor Author

rgmz commented Apr 19, 2021

I had to delete + recreate my fork; re-opened this as #329 .

@rgmz rgmz closed this Apr 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants