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

time-stats: Use label match as bar title #4

Closed
jayvdb opened this issue Aug 12, 2018 · 4 comments
Closed

time-stats: Use label match as bar title #4

jayvdb opened this issue Aug 12, 2018 · 4 comments

Comments

@jayvdb
Copy link

jayvdb commented Aug 12, 2018

@coala uses labels like type/* , difficulty/*, size/*.

When I do a time-stats.html , the prefixes are shown in the bar titles.

When I use a regex like ^difficulty/(.*)$, I would expect that only the part after the / would be the bar title.

https://hewlettpackard.github.io/yoda/yoda-time-stats.html?owner=coala&repolist=coala,coala-bears,coala-quickstart,coala-vs-code,community,corobo,gh-board&startdate=2018-04-01&labelsplit=%5Edifficulty/(.*)$&title=Closed%20issues%20during%20GSoC%20by%20difficulty&count=closed&draw=true

@jayvdb jayvdb changed the title Use match as bar time-stats: Use label match as bar title Aug 12, 2018
@jens-markussen
Copy link
Contributor

Sorry for not responding earlier. Somehow E-mail notifications went to my spam folder. Will consider and respond soon.

@jens-markussen
Copy link
Contributor

jens-markussen commented Oct 8, 2018

Hi @jayvdb. The label split field is working as intended. It selects the relevant labels using the regular expression. Your issue here is that the labels are quite long, and you would like to use only part of them for the presentation part.

I'll add labels matching an Enhancement request, and we will consider for upcoming Yoda 2 release at medium priority.

@jens-markussen
Copy link
Contributor

Ok, we decided to have a quick look at this one in detail. It turns out to be not so difficult to support positional parameter 1 in the RegExp as the legend text.

Try again. Your ^difficulty/(.*)$ example should work.

If succesfully tested, we can close this one.

@jens-markussen
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants