-
Notifications
You must be signed in to change notification settings - Fork 89
feat: Add Calendar_Generator_YearWise project (Issue No : #106) #109
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
base: main
Are you sure you want to change the base?
feat: Add Calendar_Generator_YearWise project (Issue No : #106) #109
Conversation
We're delighted to have your pull request! Please take a moment to check our contributing guidelines and ensure you've filled out the PR template for a smooth process. We will review it soon. |
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.
Pull Request Overview
Adds a new GUI calendar application project and its documentation.
- Introduces a Python/Tkinter calendar generator script.
- Provides README with setup, usage, and customization details.
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.
File | Description |
---|---|
README.md | Added project overview, usage instructions, and metadata |
Calendar.py | Implemented main GUI and calendar display logic |
Comments suppressed due to low confidence (2)
Projects/Calendar_Generator_YearWise/Calendar.py:66
- [nitpick] Variable names should follow snake_case; rename 'Show' to something like 'show_button' for consistency and clarity.
Show = Button(root, text = "Show Calendar", fg = "Black", bg = "Light Green", command = showCal)
Projects/Calendar_Generator_YearWise/Calendar.py:69
- [nitpick] Rename 'Exit' to 'exit_button' or similar to follow snake_case conventions and avoid shadowing the built-in exit function.
Exit = Button(root, text = "Exit", fg = "Black", bg = "Light Green", command = exit)
new_window.geometry('550x600') | ||
|
||
# get method returns current text as string | ||
fetch_year = int(year_field.get()) |
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.
Converting the input directly to int may crash on invalid input; consider adding a try/except block to handle non-integer values gracefully.
Copilot uses AI. Check for mistakes.
--- | ||
## How to Run | ||
``` | ||
python calendar_app.py |
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.
The README references 'calendar_app.py', but the actual script file is named 'Calendar.py'. Update the command to 'python Calendar.py' to match the file name.
Copilot uses AI. Check for mistakes.
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Calendar Application
A simple GUI calendar application built with Python and Tkinter that displays the calendar for any given year.
Features
-Easy year input
Requirements
How to Use
-A new window will open displaying the calendar for the entered year
-Use the "Exit" button to close the application
Code Structure
The application consists of:
Main window with input field
Calendar display window that opens when showing a calendar
Simple error handling for year input
How to Run
Customization
You can easily customize:
Future Improvements
License
This project is open source and available under the MIT License.