-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
[Concept Entry] C: Static Variables #5982
base: main
Are you sure you want to change the base?
Conversation
I've signed the CLA, and my account has an email associated to it. But still, it doesn't recognise the signature! |
minor fixes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for contributing to Codecademy Docs @deedend 😄
The entry looks good for a second round of review! 🚀
It looks like you've done a commit on your local device using a local id, so an email related to that would be required here. Can you please try signing the CLA with the local id |
Once the CLA is signed, we can merge the entry @deedend |
I'm sorry but I have no idea how to do that. I have my email set up in GitHub, how can I see what email is associated with the commit? |
Where can I see that id? And how can I use it to sign the CLA? |
Hey @Sriparno08, @daksh10110, and @avdhoottt, can any of you help @deedend understand where the local email ID can be found? It seems that the initial commit has been made through that. |
Hey @deedend, run the following command while on the git log db4b698 Here, After running the above command, the details for the commit is shown. Check out the |
Hey @
Hey @deedend did this work? |
Hello @mamtawardhani , sorry I've been flooded as of late, and I did the commit on my home computer so I have to take the time to deal with that when I'm at home. I will try as soon as possible. Thanks |
Hey @deedend that's understandable, but could you provide us with a deadline for this? Since its been 3 weeks to this, and we need to close it this week max. Thank you! 😄 |
Sorry, I'll try to do it tonight. If I fail, I might let it go without this thing (which I don't know what it is) |
OK, it should be good now, please confirm |
Thank you, @deedend, for your efforts, and thank you, @Sriparno08, for your quick help! 😄 @Radhika-okhade you can merge this PR now 😃 |
Description
Issue Solved
Closes #5955
Type of Change
Checklist
main
branch.Issues Solved
section.