Skip to content

Conversation

@lkishalmi
Copy link
Contributor

Just did the necessary modifications for the import. Changing license headers, and include the module into the build chain.

Right now the module automatically switches to Dark Metal on install, also the java level can be moved from 6 -> 8. I think that could be dealt with later when this "historical" import PR got merged.

@lkishalmi lkishalmi added this to the 11.3 milestone Nov 23, 2019
@lkishalmi
Copy link
Contributor Author

image
image

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.

Great.

@jkovalsky jkovalsky self-requested a review November 23, 2019 15:50
Copy link
Contributor

@jkovalsky jkovalsky left a comment

Choose a reason for hiding this comment

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

I would only suggest to change HTTP protocol to HTTPS in all links to Apache License terms i.e. https://www.apache.org/licenses/LICENSE-2.0 otherwise I like to work!

@lkishalmi lkishalmi changed the title Import Dark Metal and Dark Nimbus LaF from Oracle Donation 4 [NETBEANS-3427] Import Dark Metal and Dark Nimbus LaF from Oracle Donation 4 Nov 23, 2019
@lkishalmi lkishalmi merged commit aee8f2a into apache:master Nov 23, 2019
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