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

add command line arg delimiter for custom delimiter instead of tab. #662

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

iksi4prs
Copy link

Motivation
Adding support to parse the output of issues in plain mode,

Current problem
Current delimiter in plain mode is TAB (\t), but TAB is also used for padding the columns.
If the an issue has a value which requires more than one TAB for padding, eg where issue type is of "improvement",
then some columns will contain more than one TAB, hence not possible to parse in simple way.

Change:
Add command line arg delimiter for custom delimiter instead of tab supported only for plain mode for issues.
For backward compatibility and reduce risk of affecting other code, this new command line arg is applicable only for plain mode.

@ankitpokhrel
Copy link
Owner

Hi @iksi4prs, thank you for working on this. Would you be able to fix/update the tests? Thanks

@iksi4prs
Copy link
Author

@ankitpokhrel , will do my best. There are few old opened PRs (from other people), hence I thought this project is not active anymore.

@jcutrell
Copy link

Is this still active? I'd really like to have the delimiter option to do some analysis on the issue list.

@iksi4prs
Copy link
Author

@jcutrell I will do my best to fix the tests asap, so pr can be merged.
in any case, u can take my branch and build it, the feature itself works,
it just that uni-tests were not updated.

@iksi4prs
Copy link
Author

iksi4prs commented May 29, 2024

@ankitpokhrel , I've fixed the unit tests and added new one for this feature. please check when u have time. 10x.

@electriquo
Copy link

This is much needed feature.
@ankitpokhrel Could you please share whether you have any intention on merging/implementing this?

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.

4 participants