-
Notifications
You must be signed in to change notification settings - Fork 341
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
Migrate from numeric primitive times to Instant/Duration #49
Comments
Hi, I am new to open source and would love to help out. Could you provide me with more information regarding where in the codebase these changes need to be made? |
@emichael any further information? i am done with cloning the repo which classes do I have to work on? |
@emichael Please assign it to me |
/assign |
Hi, Asfar and Chris. Thanks so much for the offer to help! This is really something I want someone who's already done the labs and is familiar with the codebase to tackle, though. |
Ahh that's really sad. I saw the good first issue tag and i wanted to contribute. If u get some time u can explain me code base for a while. |
@emichael if there's any other issue i can work on please let me know thanks |
We should be using
Instant
andDuration
from thejava.time
package instead of millisecond times stored aslong
s orint
s.The text was updated successfully, but these errors were encountered: