Skip to content

Conversation

@kanielc
Copy link
Contributor

@kanielc kanielc commented Dec 6, 2014

These two files were missing headers.
There is other work on-going to automate updating of the headers, but
that likely won't crossover with this.

These two files were missing headers.
There is other work on-going to automate updating of the headers, but
that likely won't crossover with this.
@scala-jenkins scala-jenkins added this to the 2.11.5 milestone Dec 6, 2014
Copy link
Contributor

Choose a reason for hiding this comment

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

I guess the three blank lines are a signal to your alien conspirators that they should destroy the Earth in exactly three days.

That gives you plenty of time to repair your craft which was damaged on entry.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

They were the same blank lines in native.scala, from which I took this header.
Easy to remove if you so wish it.

@adriaanm
Copy link
Contributor

adriaanm commented Dec 9, 2014

Thanks, looks like I'll have to hold off until 2015 to merge to avoid the lawyers tracking us down over the holidays :-)

@adriaanm adriaanm modified the milestones: 2.11.5, 2.11.6 Dec 11, 2014
@retronym
Copy link
Member

LGTM

retronym added a commit that referenced this pull request Jan 22, 2015
SI-5817: Add header to language.scala and languageFeature.scala
@retronym retronym merged commit 89cf671 into scala:2.11.x Jan 22, 2015
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.

5 participants