-
Notifications
You must be signed in to change notification settings - Fork 689
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
Update shading_language_selection.cpp #1260
base: main
Are you sure you want to change the base?
Conversation
Fix typo in a LOGI message
|
@RobertHenry6bev This fails at CI. You need to update the copyright of the file(s) you changed. |
Update copyright dates
Hi @RobertHenry6bev, it looks like you need to sign the CLA before we can merge this. LMK if questions on how to do that. Thanks. |
As far as I remember, I have never done this before. (The overhead is a little high for a small change, but, ok). So I vist https://cla-assistant.io/my-cla and after granting the assistant all the permissions it asks for, am led to a web page that says (with my hand edits to insert table col separators '|': Status Repository CLA Date of Signature Is this sufficient? |
Thanks. Yah, does seem like overkill for such a small change, but the tools are strict on licensing around here.. Looks like there's a merge conflict now (hopefully easy to resolve?). If this is too much of a pain, just LMK - we're happy to do this tweak ourselves too:) |
Yah, this is too much of a pain for me right now. I would appreciate it if you closed this PR and then applied the changes yourself. Thank you very much! |
Sounds good. We'll take care of it. Thanks for pointing this out to us - very much appreciate the help! |
Fix typo in a LOGI message
Description
Fix typo
I have done no testing. I only see this typo in one place.