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

ci: adds a GH action workflow file for F5 CLA automation #224

Merged
merged 7 commits into from
Apr 26, 2024

Conversation

TasoOneAsia
Copy link
Member

@TasoOneAsia TasoOneAsia commented Mar 13, 2024

Proposed Changes

Adds a GitHub action workflow file configured for F5 contributor license agreement automation and compliance. This action is configured to target a shared signature store for F5 Open Source projects. As part of the configuration, the following GitHub usernames were whitelisted as maintainers:

  • 4141done
  • dekobon

Any contributors not included in this whitelist will be asked to review the F5 CLA upon a pull request to the master main branch and to leave a comment agreeing to the terms before their contribution is merged.

Let me know if there are any other questions :)

@TasoOneAsia TasoOneAsia requested a review from 4141done March 13, 2024 16:06
@TasoOneAsia TasoOneAsia self-assigned this Mar 13, 2024
@alessfg alessfg changed the base branch from master to main April 25, 2024 21:30
@alessfg alessfg self-assigned this Apr 25, 2024
@alessfg
Copy link
Collaborator

alessfg commented Apr 26, 2024

It just occured to me that we might want to update the contributing guide and PR template to note that F5 now requires a CLA. I'll work on it as soon as I get a chance.

@TasoOneAsia
Copy link
Member Author

@alessfg feel free to reference my pr to f5/unovis which included changes to CONTRIBUTING.md. Though, looking back on that patch, there are definitely a couple missing commas.

@alessfg alessfg force-pushed the ci/f5-cla-automation branch from 8079ed3 to cdce8a3 Compare April 26, 2024 19:31
@alessfg alessfg force-pushed the ci/f5-cla-automation branch from cdce8a3 to 7351cbb Compare April 26, 2024 19:32
* add CLA to the PR template
* mention CLA in the contributing guidelines
@alessfg alessfg added the dependencies Pull requests that update a dependency file label Apr 26, 2024
@alessfg alessfg force-pushed the main branch 2 times, most recently from 6d2bced to 119c0e0 Compare April 26, 2024 21:15
@alessfg alessfg merged commit 421e963 into nginxinc:main Apr 26, 2024
8 checks passed
Copy link

CLA Assistant Lite bot: 🎉 Thank you for your contribution. It appears you have not yet signed the F5 Contributor License Agreement (CLA), which is required for your changes to be incorporated into an F5 project. Please kindly read the F5 CLA and comment the following to agree:


I have hereby read the F5 CLA and agree to its terms


1 out of 2 committers have signed the CLA.
✅ (TasoOneAsia)[https://github.com/TasoOneAsia]
@alessfg
You can retrigger this bot by commenting recheck in this Pull Request

@alessfg alessfg added github_actions Pull requests that update GitHub Actions code and removed dependencies Pull requests that update a dependency file labels May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
github_actions Pull requests that update GitHub Actions code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants