RepoSense icon indicating copy to clipboard operation
RepoSense copied to clipboard

Support datetime for since and until repo-config.csv

Open joeng03 opened this issue 8 months ago • 3 comments

Currently, we have #2369 to support datetime for since and until through CLI. Due to urgency for the CLI functionality to be merged, let's move support for datetime in repo-config.csv as another separate issue, as it may contain significant test and system test changes, as well as User Guide updates.

joeng03 avatar Apr 02 '25 13:04 joeng03

If possible, describe the solution

In terms of the function code the only thing should be changing is the one in the RepoConfigCsvParser.java. Currently, the since date and until date are taken in the format of dd/MM/yyyy and then hardcoded into valid LocalDatetime object by manually adding the default since date start time and default until date end time.

A way to solve this issue is to relax the constraint on the format of date time input in the CSV. We can support both dd/MM/yyyy and dd/MM/yyyyTHH:mm at the same time. Only hardcoding and overwriting the dates where specific time is not provided.

It will be suitable for first-comer to work on this so that they can get a rough idea of how the arguments are parsed and the repos are analyzed from the experience of implementing this one.

CYX22222003 avatar Apr 08 '25 05:04 CYX22222003

Hi! Could I be assigned this issue please? Thanks!

gabriellegtw avatar Jun 16 '25 07:06 gabriellegtw

@gabriellegtw you can send a PR for this issue. There is no need to (and not possible to, as you are not a member of this GitHub organisation) assign the issue to you firstjk.

damithc avatar Jun 16 '25 12:06 damithc