Skip to content

Conversation

@Anjalisharma8072
Copy link
Contributor

this PR includes replacement of () with [] as When accessing the environment variable, it should use os.environ['password'] instead of os.environ('password').

@intelagense
Copy link
Collaborator

This looks good. There two more copies of the typo in automate-secret-santa-emails-with-smtp.mdx as well. Can you update these in this same branch so we can knock out everything at once? 🙏

@Anjalisharma8072
Copy link
Contributor Author

I am sorry but i only find one copy of it and additionally also added the missing comma

@intelagense
Copy link
Collaborator

intelagense commented Oct 29, 2024

Oh sorry, I thought this was the solution file! I meant two in the MDX, and one more in the actual solution file. 😭
Do you mind adjusting the solution file as well? 🙏

@Anjalisharma8072
Copy link
Contributor Author

done :)Now check it!

@intelagense
Copy link
Collaborator

Thanks! Merging now!

@intelagense intelagense merged commit 55f4a77 into codedex-io:main Oct 31, 2024
@intelagense intelagense added the hacktoberfest-accepted https://hacktoberfest.com/ label Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hacktoberfest-accepted https://hacktoberfest.com/

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants