Skip to content

Create date_and_time.md #17

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.

Sign up for GitHub

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

Merged

AdityaJ7 merged 2 commits into Python-World:master from vanigupta20024:master

Oct 9, 2020

Merged

Create date_and_time.md #17

AdityaJ7 merged 2 commits into Python-World:master from vanigupta20024:master

Oct 9, 2020

Conversation

Copy link

Contributor

@vanigupta20024 vanigupta20024 commented Oct 9, 2020

Description

Created snippet to show current date and time in python and explained as well.

A short summary of what is included in your Pull Request.

Fixes #16

Type of change

Choose one or more options from the following as per the nature of your Pull request.

NOTE: These boxes can be checked using [X]
Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Documentation Update

Checklist:

Please tick all the boxes that are fulfilled by your Pull Request.

I have named my files and folder, according to this project's guidelines.
My code follows the style guidelines of this project.
My Pull Request has a descriptive title. (not a vague title like Update index.md)
I have commented on my code, particularly in hard-to-understand areas.

@vanigupta20024


 Create date_and_time.md

336a822

@AdityaJ7 AdityaJ7 self-requested a review

October 9, 2020 07:30

AdityaJ7 reviewed

View reviewed changes

snippets/python/date_and_time.md Outdated

# now() method returns current datetime object

cur_time = datetime.datetime.now()

print("Date: ", cur_time.day, "/", cur_time.month, "/", cur_time.year)

Copy link

Member

@AdityaJ7 AdityaJ7 Oct 9, 2020 •

edited

Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use f strings to make it more pythonic
you can use this
print(f"Date: {cur_time.day} / {cur_time.month} / {cur_time.year}")

AdityaJ7 reviewed

View reviewed changes

snippets/python/date_and_time.md Outdated

cur_time = datetime.datetime.now()

print("Date: ", cur_time.day, "/", cur_time.month, "/", cur_time.year)

print("Time: ", cur_time.hour, "h:", cur_time.minute, "m:", cur_time.second, "s")

Copy link

Member

@AdityaJ7 AdityaJ7 Oct 9, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use f-strings here as well

@AdityaJ7 AdityaJ7 added the hacktoberfest-accepted label

@vanigupta20024


 Update date_and_time.md

18c2049

@vanigupta20024

Copy link

Contributor Author

vanigupta20024 commented Oct 9, 2020

Done! Thanks for letting me know this, did not know about is earlier!

Copy link

Member

AdityaJ7 commented Oct 9, 2020

Done! Thanks for letting me know this, did not know about is earlier!

No problem.

Nice job with the snippet 👍🏽

AdityaJ7 approved these changes

View reviewed changes

@AdityaJ7 AdityaJ7 merged commit 97b3d10 into Python-World:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hacktoberfest-accepted

2 participants

@vanigupta20024 @AdityaJ7