Add support for jmeter-plugins and fancy graph generation #51
Conversation
generation. For now it generates all the reports. - Bumped version number.
existing project declares the plugins we will remove the in built ones.
|
@pokowaka There are a whole bunch of whitespace/formatting changes in this PR, should they be a separate PR instead of being interspersed with functional changes? |
|
Probably intellij going to town with the source. Let me see if I can just fix the whitespace. |
|
I like this commit but do you think there should be an option to turn off the loading of the ext jmeter-plugins just in case someone wanted to use the bare-bones jmeter for some reason? Also, why did you choose not to load extras-libs or hadoop ?? When this update takes, I will update my project to reflect the update:https://github.com/djangofan/launch-jmeter |
|
@djangofan, that might be better for backwards compatibility as well. |
|
Can you put an update in the docs and/or wiki for the changes you make? If you are loading the external plugins, it might be nice to mention that in the main page. |
|
@djangofan i will update the documentation. |
- Make the extension optional, so the optional jars are only added when needed. - Undid some layout changes. - Updated documentation.
|
@djangofan, @kulya, @foragerr any of you had a chance to look at the suggested changes? |
|
Looks good to me. Just so you know, only @kulya has rights to approve a PR. |
|
It looks like @kulya is not very active in this repo. If nothing happens in the following week I will look into hard forking and pushing it to a maven repo under a different name. |
|
I think if you just hang in there and be patient, it is likely that @kulya will check-in and look at your pull request. It could take a few months, but just hang in there. |
|
One last ditch ask to get this merged and released, I'm really not looking for ward to forking this plugin :( |
|
@kulya They just released version 1.3.0 of the Jmeter plugins. That being the case, maybe we will see some movement on this? The reason being that the jmeter-gradle-plugin needs to be tested with and perhaps versioned for 1.3.0 . http://jmeter-plugins.org/wiki/Changelog/ |
|
Merged and forked here: https://github.com/jmeter-gradle-plugin/jmeter-gradle-plugin |
No description provided.