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

Script for "Weather Notifier" added #963

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

Davda-James
Copy link

@Davda-James Davda-James commented Oct 2, 2024

Weather Notifier -

The Weather Notifier script is a Python-based application that fetches and displays real-time weather information for a specified city. The script uses the OpenWeatherMap API to retrieve weather data and presents it through a user-friendly GUI built with Tkinter. Additionally, it supports both manual and automatic weather updates, allowing users to either fetch the weather data on demand or set a recurring interval for automatic updates.

It solves the issue #954 -

Self Checks

  • One Change in one Pull Request
  • My file is in proper folder (Name of folder should be in lowercase with no space in between) (weather_notifier/weather_notifier.py)
  • I am following clean code and Documentation and my code is well linted with flake8.
  • I have added README.md and requirements.txt (Include version numbers too e.g. pandas==0.0.1) with my script
  • I have used REPO README TEAMPLATE (Necessary)
  • Just including required dependencies in requirements.txt
  • Added Screenshots of the gui
  • Followed the PEP8 using flake8 linter and black

I am open to suggestions if any and if not then please merge it and add the "hacktoberfest-accepted" label to it.

@Davda-James
Copy link
Author

The linting error is not in my folder, it is in the existing folder ( video_cropper and pdf_watermarkadder) and also I have tried to resolve some linting issues in another existing scripts but still there is some linting error in their folders which is causing the linker check to fail. Thank You

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