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

Fix NameError: name 'ip' is not defined #15

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gandrunx
Copy link

@gandrunx gandrunx commented Aug 8, 2024

Summary

This pull request addresses a NameError encountered when the ip variable is referenced before it is defined.

Details

  • Properly defined the ip variable to ensure it is initialized before use.
  • Updated the relevant code sections to avoid the NameError.
  • Add load_dotenv()
  • Add .env.example
  • Add .gitignore for python project
  • Fix error message (InsecureRequestWarning from urllib3)
  • Fix error message banner

Testing

  • Tested the changes locally to ensure the error no longer occurs.
  • Verified that the functionality involving the ip variable works as expected after the fix.

Related Issues

Additional Notes

  • Please review the changes and let me know if any further adjustments are needed.
  • I am open to feedback and ready to make any necessary modifications.

Thank you for considering this fix. Looking forward to your feedback.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant