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

polyglot-scala: Added default value for pom property modelversion (4.0.0) #158

Closed
wants to merge 3 commits into from

Conversation

lefou
Copy link
Contributor

@lefou lefou commented Mar 14, 2018

It is more than expected, that this version will not change in the near future.

It is more than expected, that this version will not change in the near future.
@mosabua
Copy link
Member

mosabua commented Mar 22, 2018

Merged with f765ca4

@mosabua mosabua closed this Mar 22, 2018
@mosabua
Copy link
Member

mosabua commented Mar 22, 2018

I merged this but I am getting test failures.. wanna have a look @lefou ?

@lefou
Copy link
Contributor Author

lefou commented Mar 22, 2018

Why do you dropped my commits? I see only my github name in your commit message.

@lefou
Copy link
Contributor Author

lefou commented Mar 22, 2018

@mosabua I can reproduce the failing tests. Must have overlooked it, sorry. I'll create another PR.

lefou added a commit to lefou/polyglot-maven that referenced this pull request Mar 22, 2018
This commit completes pull request takari#158
@lefou lefou mentioned this pull request Mar 22, 2018
@mosabua
Copy link
Member

mosabua commented Mar 22, 2018

I screwed up the merging and it was too painful on the other PR due to conflicts .. so I thought I just commit this normal on master .. it was just easier late at night yesterday .. sorry

@lefou
Copy link
Contributor Author

lefou commented Mar 22, 2018

Thanks for telling me. That's OK.

I'd be glad to offer my help. I think, I have a fairly good understanding of polyglot-maven and use it in several projects. With commit access I could merge approved or trivial commits directly. WDYT?

@mosabua
Copy link
Member

mosabua commented Mar 22, 2018

Sure.. you would have to sign the Takari CLA .. did you do that already?

@lefou
Copy link
Contributor Author

lefou commented Mar 23, 2018

I'm not sure, I think not. Where can I find it?

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.

2 participants