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.

Sign up for GitHub

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

Jump to bottom

Add missing file headers and update (C) year #4381

Merged
merged 2 commits into from
Dec 16, 2019
Merged

Add missing file headers and update (C) year #4381

merged 2 commits into from
Dec 16, 2019

Conversation

pzygielo
Copy link
Contributor

No description provided.

@pzygielo pzygielo mentioned this pull request Dec 13, 2019
Merged
@jbee
Copy link
Contributor

jbee commented Dec 13, 2019

jenkins test please

jbee
jbee approved these changes Dec 13, 2019
View reviewed changes
Copy link
Contributor

@jbee jbee left a comment

Choose a reason for hiding this comment

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

Thanks again, could you also update the header of HazelcastTimerStore.java that was changed in #4311?

@pzygielo
Copy link
Contributor Author

Thanks again, could you also update the header of HazelcastTimerStore.java that was changed in #4311?

Updated.
I hoped for that change to transfer from #4377 😉

@pzygielo pzygielo changed the title Add missing file headers Add missing file headers and update (C) year Dec 13, 2019
@pzygielo pzygielo requested a review from jbee December 13, 2019 09:59
@jbee
Copy link
Contributor

jbee commented Dec 13, 2019 •
edited
Loading

I hoped for that change to transfer from #4377

Missed that. It does so this was a bit unnecessary :D

@jbee
Copy link
Contributor

jbee commented Dec 13, 2019

jenkins test please

@MeroRai MeroRai merged commit 8d9a56e into payara:master Dec 16, 2019
@pzygielo pzygielo deleted the headers branch December 16, 2019 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Cousjava Cousjava Cousjava approved these changes

@jbee jbee jbee approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@pzygielo @jbee @Cousjava @MeroRai