Add --plot-to-board to plot command #184
Conversation
Ronaq13
force-pushed
the
adding_plot-to-board_in_plot
branch
from
e7f901e to
8ec340c
Compare
This flag of plot command will send the plotted burndown chart to the first card of the `Done` column. close openSUSE#134
Ronaq13
force-pushed
the
adding_plot-to-board_in_plot
branch
from
8ec340c to
d578849
Compare
| if options[:plot_to_board] | ||
| begin | ||
| chart = BurndownChart.new @@settings | ||
| chart.update(options) |
There was a problem hiding this comment.
This is also modifying the yml file, which we don't want to modify here. We only want to print the image and push it to the card.
There was a problem hiding this comment.
Please help identifying the line at which it modifying the .yml file. And by .yml file do you mean burndown-data-01.yml file which is created at time of initialization
There was a problem hiding this comment.
inside of the update method.
Yes, the burndown-data-i.yaml file, where i is the number of the sprint
There was a problem hiding this comment.
@Ana06 I dont see any change in the yml file. Please check now is it good to merge?
…g backlog, it closes issue openSUSE#45
Added the 'to' statement for the corresponding 'expect' statement in the unit test of checking the file presence. close openSUSE#178
Trollolo accepts a definition of custom doing columns in the burndown YAML file. This used to be part of the `not_done_columns` setting. This has been split up into `todo_columns` and `doing_columns`. Old configurations which use the `not_done_columns` setting will have to be adjusted. There should be an error message now explaining what to do. This fixes a bug where custom doing columns where not properly accounted for when creating burndown data.
|
@Ronaq13 @matheussbernardo @cornelius where are those commits coming from? |
|
@Ronaq13 what is the state of this PR? 🤔 |
|
Closing due to inactivity, @Ronaq13 feel free to reopen it if you continue working on it 😉 |
This flag of plot command will send the plotted burndown chart to the first
card of the
Donecolumn.close #134