Idea was to tidy some of our code and remove redundant wrapping into multiple lines
If you see some spots where you would prefer multi-lines instead of a single line, you just have to add the trailing comma back. I've quickly scanned through the files and added some of the trailing commas (mostly for our example configurations).
[x] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":
I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.
I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorized to submit this contribution on behalf of the original creator(s) or their licensees.
I certify that the use of this contribution as authorized by the Apache 2.0 license does not violate the intellectual property rights of anyone else.
I have not referenced individuals, products or companies in any commits, directly or indirectly.
I have not added data or restricted code in any commits, directly or indirectly.
Description
See following discussion thread: https://github.com/mckinsey/vizro/pull/283#discussion_r1467341688
Idea was to tidy some of our code and remove redundant wrapping into multiple lines
If you see some spots where you would prefer multi-lines instead of a single line, you just have to add the trailing comma back. I've quickly scanned through the files and added some of the trailing commas (mostly for our example configurations).
Note: Will be merged after https://github.com/mckinsey/vizro/pull/293
Screenshot
Notice
[x] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":