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

Fix jira sprint parsing #615

Open
wants to merge 2 commits into
base: develop
from

Conversation

Projects
None yet
1 participant
@campbellr

campbellr commented Oct 23, 2018

Jira doesn't escape commas in the value field of the sprint string so it's not safe to just split on all commas.

In my case, I ran into a sprint with a goal that contained a bunch of commas.

Splitting on the = character and then using rsplit to only grab the last , is maybe a bit safer.

campbellr added some commits Oct 21, 2018

Fix for jira sprint parsing bug
Jira doesn't escape commas in the value field of the sprint string so we
can't just split on all commas.

In my case, I ran into a sprint with a goal that contained a bunch of
commas.

Splitting on the '=' character and then using rsplit to only grab the
last ',' is a bit safer.
@campbellr

This comment has been minimized.

campbellr commented Oct 24, 2018

I took a look at the Travis failures, and they seem to be unrelated. On was a failing Gerrit test, the other was an error downloading from AWS...

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