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

Added replace method on Delorean object #81

Merged
merged 4 commits into from
Sep 16, 2016
Merged

Conversation

masnun
Copy link
Contributor

@masnun masnun commented Mar 11, 2016

As discussed in #80 - please review and let me know if there needs to be further modifications.

@myusuf3
Copy link
Owner

myusuf3 commented Mar 11, 2016

docs? :)

@masnun
Copy link
Contributor Author

masnun commented Mar 11, 2016

Ah yes, sorry forgot. Let me get to that.

On Sat, Mar 12, 2016 at 2:22 AM, Mahdi Yusuf [email protected]
wrote:

docs? :)


Reply to this email directly or view it on GitHub
#81 (comment).

Abu Ashraf Masnun | +8801711960803 | http://masnun.me

@masnun
Copy link
Contributor Author

masnun commented Mar 11, 2016

I added the docs. Please review.

self.assertEqual(do.replace(minute=23).datetime.minute, 23)
self.assertEqual(do.replace(second=45).datetime.second, 45)


Copy link
Owner

Choose a reason for hiding this comment

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

I would also like to see a few asserts on the entire datetime object as well and also assuring the timezone is unchanged.

@masnun
Copy link
Contributor Author

masnun commented Mar 12, 2016

I added a test showing that the timezone doesn't change. And a test using datetime arithmetic with timedelta.

@masnun
Copy link
Contributor Author

masnun commented Mar 12, 2016

And a few asserts on the datetime objects.

@masnun
Copy link
Contributor Author

masnun commented Mar 16, 2016

Is there anything else you would like me to help with?

@myusuf3
Copy link
Owner

myusuf3 commented Mar 16, 2016

I just need a moment to look it over. I will try to get to it before this weekend.

@masnun
Copy link
Contributor Author

masnun commented Mar 30, 2016

Hi, just checking how it looks. Did you manage to get a look at it?

@myusuf3 myusuf3 merged commit abeed71 into myusuf3:master Sep 16, 2016
@myusuf3
Copy link
Owner

myusuf3 commented Sep 16, 2016

Awesome thanks for contribution. Sorry for my terrible delay.

@masnun
Copy link
Contributor Author

masnun commented Sep 16, 2016

No problem :-)

On Friday, September 16, 2016, Mahdi Yusuf [email protected] wrote:

Awesome thanks for contribution. Sorry for my terrible delay.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#81 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AALsN-wSRKxcldTXL96DDQIdpBap_xOoks5qqsyagaJpZM4Hu7w_
.

Abu Ashraf Masnun | +8801711960803 | http://masnun.me

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.

2 participants