RepoSense icon indicating copy to clipboard operation
RepoSense copied to clipboard

Support different time periods for different repos for report-config.yaml

Open lyuanww opened this issue 8 months ago • 4 comments

What feature(s) would you like to see in RepoSense

With the the one-stop config feature completed (PR: #2299) , we can enhance the feature by supporting different time periods for different repos for more customisability.

Optimally, the since-date and until-date fields should be optional.

We have already implemented this feature (refer to #2280) for the CSV dates.

Additional context

Will need to take note of resolving conflict for the dates between different configurations (CLI, CSV and report-config.yaml)

Please refer to #2334 for more context.

lyuanww avatar Apr 19 '25 17:04 lyuanww

Hi, could i work on this issue? Thanks

lihongong avatar Jul 19 '25 07:07 lihongong

@lihongong

Sure, we'll be happy to have your help.

If you need info about the setup, workflow, PRs etc, you can check out our developer guide.

Please make sure closely follow the workflow when making contribution.

However, this task may be a bit challenging as it needs to align with the conflict resolving mechanisms for dates across different configurations. Please refer to the PR #2280 and issue #2334 for more context before making any breaking changes

CYX22222003 avatar Jul 19 '25 15:07 CYX22222003

Hi @CYX22222003 , in the UG configFiles.md, it says that repo-config.csv supports date in the format of • DD/MM/YYYY • DD/MM/YYYY'T'HH:mm • DD/MM/YYYY'T'HH:mm:ss However, it only supports DD/MM/YYYY when I run it in my computer. Should I support all 3 formats in report-config.yaml and fix the implementation of RepoConfigCsvParser as well?

lihongong avatar Jul 27 '25 16:07 lihongong

Hi @CYX22222003, sorry for the late comment, please review my PR here. Thank you.

lihongong avatar Aug 07 '25 13:08 lihongong