-
- Notifications
You must be signed in to change notification settings - Fork 263
City weather #147
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
City weather #147
Conversation
Please add a screenshot in readme file @Komal-99 |
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.
Changes needed:
- The program file should have comments wherever needed.
- The readme file should be named as README.md
- Elaborate the README of your project - purpose, use, steps of the workflow, compilation process, output screenshot etc. There is README template to follow in README file of the project
- The project folder should be titled as "Weather Notifier" as you had mentioned in the issue.
- The screenshot you add should be in "Images" folder.
Do the needful changes. @Komal-99
okay @prathimacode-hub I am unable to add images in images folder while trying to upload it is showing site crash try again |
…er Notifier/README.md
…cripts/Weather Notifier/weather.py
…s/Weather Notifier/requirements.txt
Changes to be done: Ok, the PR commit message is not in template format yet. Do the needful changes. @Komal-99 |
@prathimacode-hub |
Read my earlier reply to gain clarity of why it isn't uploading. I suggested you not to use "user-images". Instead use the screenshot uploaded in the repo. If you had followed right format, it should have been solved. @Komal-99 @Debashish-hub Kindly look into this issue and guide. |
I checked your reply and tried in similar way . but don't know why this error is occurring the screenshot i added in readme earlier i was using same not user-images |
Just create an image folder and add your screen shots into it. What's the problem you are facing @Komal-99 |
That's because they were not a part of LGMSOC program. This is to be fair with everyone. It isn't really tough. We can help you out. Just let me know of how you're following or trying to add. So that we can suggest accordingly. @Komal-99 |
Why did you closed it @Komal-99 |
Creating New one |
Related Issue
Info about the related issue
LGM-SOC'21 Participant
Contributor
Closes: #120
Describe the changes you've made
Give a clear description what modifications you have made
Added REAME.md
Added requirements.txt
Added weather.py
Added Images
Type of change
What sort of change have you made:
How Has This Been Tested?
Describe how have you verified the changes made
Checklist:
Screenshots