Skip to content

[NETBEANS-1161] Move java modules to java subdirectory#745

Merged
geertjanw merged 6 commits intoapache:masterfrom
ralphbenjamin:NETBEANS-1161_move_java_modules_to_java_subdirectory
Aug 22, 2018
Merged

[NETBEANS-1161] Move java modules to java subdirectory#745
geertjanw merged 6 commits intoapache:masterfrom
ralphbenjamin:NETBEANS-1161_move_java_modules_to_java_subdirectory

Conversation

@ralphbenjamin
Copy link
Contributor

No description provided.

@geertjanw
Copy link
Member

Wow, that's fast work. :-) Thanks @ralphbenjamin. (Feel free to take on the ide cluster next.)

@ralphbenjamin
Copy link
Contributor Author

Thanks @geertjanw! But before I pickup anything else, it seems from the build I'm not done yet with the java cluster :-)

@mcdonnell-john
Copy link
Contributor

This might be what's wrong with my 2 prs will take a look tonight

@geertjanw
Copy link
Member

If this results in the build succeeding, should we merge this pr, or is there more to be done here?

@geertjanw
Copy link
Member

OK, everything is green, great job, any reason not to merge this?

@ralphbenjamin
Copy link
Contributor Author

@geertjanw I think someone should review the commits before merging.

@geertjanw
Copy link
Member

What specifically would you like to be reviewed, which parts are you most unsure about?

Copy link
Member

@geertjanw geertjanw left a comment

Choose a reason for hiding this comment

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

Thanks. Approved.

@geertjanw
Copy link
Member

Awesome, merging this.

@geertjanw geertjanw removed the request for review from Chris2011 August 22, 2018 14:28
@geertjanw geertjanw merged commit accdbad into apache:master Aug 22, 2018
@ralphbenjamin ralphbenjamin deleted the NETBEANS-1161_move_java_modules_to_java_subdirectory branch August 22, 2018 15:15
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.

3 participants