Skip to content

Netbeans license headers cleanup check-in #1#2281

Merged
junichi11 merged 1 commit intoapache:cndfrom
BradWalker:cnd_cleanup_license_headers_1
Jul 31, 2020
Merged

Netbeans license headers cleanup check-in #1#2281
junichi11 merged 1 commit intoapache:cndfrom
BradWalker:cnd_cleanup_license_headers_1

Conversation

@BradWalker
Copy link
Copy Markdown
Member

Netbeans license headers cleanup check-in #1.. The first of several..

Add missing ALv2 license headers to form files.

Add missing ALv2 license headers to form files.
@junichi11
Copy link
Copy Markdown
Member

@BradWalker #xx means issue/pr number in the GitHub. So, should use another one.

Copy link
Copy Markdown
Contributor

@matthiasblaesing matthiasblaesing left a comment

Choose a reason for hiding this comment

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

Looks sane to me. @junichi11 I don't think the "#1" in the commit message is a deal breaker. That github decided to try to gather information from plain text insteaf of structured information is a problem of github, not Brad. Try to add some random "@" into messages and a random number of people get CCed.

In any case, @BradWalker will need a committer to commit this, so please indicate if you require an updated commit message, commit this yourself, or want me to merge it.

@junichi11
Copy link
Copy Markdown
Member

@matthiasblaesing

I don't think the #1 in the commit message is a deal breaker. That github decided to try to gather information from plain text insteaf of structured information is a problem of github, not Brad. Try to add some random "@" into messages and a random number of people get CCed.

I didn't mean that. Actually, I approved this. I just would like to avoid unnecessary notification/link. I meant I suggest use another one (from next time; It's OK with me to merge this as it is) if we need not use #xx absolutely. It's a problem of GitHub, but it's better if we can avoid that, I think.

@BradWalker
Copy link
Copy Markdown
Member Author

Hey @junichi11 and @matthiasblaesing, thanks for the feedback.. I would NEVER have known that GitHub was trying to parse my pull request for notification.. Thanks.. I'll be more careful next time..

Can @junichi11 commit my changes for me?

Thanks to both of you!

@junichi11
Copy link
Copy Markdown
Member

@BradWalker Never mind :) Merging.

@junichi11 junichi11 merged commit cfd469d into apache:cnd Jul 31, 2020
@matthiasblaesing
Copy link
Copy Markdown
Contributor

Thanks to the both of you. 👍

@BradWalker BradWalker deleted the cnd_cleanup_license_headers_1 branch August 2, 2020 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants