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.

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

Remove native-image.properties from micrometer-core #4246

Merged
merged 1 commit into from
Oct 20, 2023

Conversation

zakkak
Copy link
Contributor

@zakkak zakkak commented Oct 16, 2023

Closes #4245

@pivotal-cla
Copy link

@zakkak Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@zakkak
Copy link
Contributor Author

zakkak commented Oct 16, 2023

@pivotal-cla I am a Red Hat employee and should be covered by a corporate CLA. Additionally this might be an Obvious Fix.

@shakuzen
Copy link
Member

@zakkak For corporate CLAs to be recognized by the CLA bot, it depends on whether your company's corporate CLA was done via a GitHub org or a corporate email domain. Depending on which, you'll need to either associate a RedHat email address with this GitHub account or make your membership public to the RH org that was used to sign the corporate CLA so the bot can see it. See the corporate section of the CLA bot FAQ: https://cla.pivotal.io/about

@zakkak
Copy link
Contributor Author

zakkak commented Oct 16, 2023

@shakuzen AFAIK there is no redhat organization on github and the CLA was probably done with the corporate email domain. I have my redhat email associated with my github profile and am also using it in my commits.

It looks like @DanHeidinga had similar issues in the past (see #2514). Dan do you recall how you got this working?

@DanHeidinga
Copy link
Contributor

It looks like @DanHeidinga had similar issues in the past (see #2514). Dan do you recall how you got this working?

I think I had an email exchange with "cla@pivotal.io" but I don't have access to those email anymore to confirm.

@shakuzen
Copy link
Member

@pivotal-cla This is an Obvious Fix

@pivotal-cla
Copy link

@zakkak This Pull Request contains an obvious fix. Signing the Contributor License Agreement is not necessary.

@shakuzen
Copy link
Member

I've marked this as an obvious fix - there isn't another way we could go about achieving this and it is deleting a one line (other than license) metadata file. Thank you for the contribution.

If you would like to pursue making sure the CLA bot recognizes you as covered by a corporate CLA, I would recommend reaching out to someone internally at RedHat that deals with corporate CLAs to get instructions from them on what you are supposed to do to contribute to projects under the "Spring" Pivotal CLA. I don't remember exactly what was done when this came up 3 years ago for Dan's contribution, but he does still show as fulfilling CLA requirements.

@shakuzen shakuzen merged commit 0ec2909 into micrometer-metrics:main Oct 20, 2023
6 checks passed
@zakkak zakkak deleted the 2023-10-16-fix-4245 branch October 20, 2023 10:30
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.

Remove native-image.properties from micrometer-core
4 participants