Skip to content

Conversation

@Davda-James
Copy link
Contributor

@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
Contributor 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

@Davda-James
Copy link
Contributor Author

Now my script has no merge conflicts, thanks for resolving the issue of linting by updating version and actions. Please merge this script . Thank You .

@pawangeek pawangeek linked an issue Oct 12, 2024 that may be closed by this pull request
1 task
@pawangeek pawangeek added hacktoberfest Supporting Hacktoberfest hacktoberfest-accepted Supporting completion of hacktober fest labels Oct 12, 2024
@pawangeek pawangeek merged commit c3902cf into python-geeks:main Oct 12, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hacktoberfest Supporting Hacktoberfest hacktoberfest-accepted Supporting completion of hacktober fest

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Weather notifier 🌤️

2 participants