i dont think pep8 will accept this :)

you have made this change in each and every commit of yours

@smitthakkar96 can you please cf : https://pagure.io/pagure/pull-request/1936#comment-18995

you shall remove these lines instead of commenting them :)

Why do you want to remove them in the first place?

And my question highlights how you commit message needs to be adjusted so that someone understand what it is about without having to actually go and look at this ticket to understand the issue that is being resolved.

As mentioned inline, the commit message needs to be re-worded to be self explanatory.

In addition, it would be good to have unit-tests for this :)

Ok thanks @pingou I will look into it and try to add tests.

@smitthakkar96 what is the progress on this ?

@farhaan I am travelling so I will need to work on it when I reach home.

@pingou I will reword the commit message but I have removed those two lines as they were the causing the program to store the date of issue when it was created so I removed those lines, now by default the date and time of action will be stored as in models we have set default=datetime.datetime.utcnow() .

@smitthakkar96 I realized this when reading the ticket, and this shows why the commit message needs to be reworded :)

@smitthakkar96 progress on this ?

ping?

@vivekanand1101 re-did this PR in https://pagure.io/pagure/pull-request/2072 so we could merge it :)

Thank you both!