p4lang / pna

Portable NIC Architecture
Apache License 2.0
55 stars 21 forks source link

Fix misspelling of definitions #56

Closed apinski-cavium closed 1 year ago

apinski-cavium commented 1 year ago

Just a small misspelling of definitions in the spec which should be fixed.

Signed-off-by: Andrew Pinski apinski@marvell.com

onf-cla-manager[bot] commented 1 year ago

Hi @apinski-cavium, this is the ONF bot 🤖 I'm glad you want to contribute to our projects! However, before accepting your contribution, we need to ask you to sign a Contributor License Agreement (CLA). You can do it online, it will take only a few minutes:

✒️ 👉 https://cla.opennetworking.org

After signing, make sure to add your Github user ID apinski-cavium to the agreement.

For more information or help:" https://wiki.opennetworking.org/x/BgCUI

jfingerh commented 1 year ago

@apinksi-cavium I am guessing that the ONF bot is probably complaining for reasons that should be unrelated to you actually having a signed CLA on file? IIRC, you have gotten PRs into the spec and p4c before.

apinski-cavium commented 1 year ago

@jfingerh I have not signed the CLA just yet (waiting on approvals inside Marvell; should be done by the end of the month). But I assumed the changes I have done so far really don't have huge copyright issues as most are small enough not even to be copyrightable which is why they have been accepted.