pzygielo commented on pull request #107: URL: https://github.com/apache/maven-scm/pull/107#issuecomment-974698770
> I think we should merge this and rename the ticket to make it explicitly clear that this plugin is _not_ threadsafe. Goals *are* already marked with threadSafe=false, there is no third option. It would not make it more not thread safe. It'd be _explicitly clear_ only in the code, so interested party would need to check it in the source. I don't like it. It would have no impact on the build itself. Perhaps plugin-plugin could add this to the site, like is done for other > Attributes: > > Requires a Maven project to be executed. > Executes as an aggregator plugin. This would as well save from analysis and changes to _actually_ make the goals thread safe. Hence I'm :-1: for merging. <sup>(But I'm not maintainer here, so only sharing my opinion.)</sup> -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
