Branch: refs/heads/release/1.x

  Home:   https://github.com/jenkinsci/stashnotifier-plugin

  Commit: adbfc5c4fcf24a25f5c3abc49a9b07ea826c123e

      
https://github.com/jenkinsci/stashnotifier-plugin/commit/adbfc5c4fcf24a25f5c3abc49a9b07ea826c123e

  Author: René Scheibe <rene.sche...@gmail.com>

  Date:   2020-01-05 (Sun, 05 Jan 2020)



  Changed paths:

    M pom.xml

    M src/main/resources/index.jelly

    M 
src/main/resources/org/jenkinsci/plugins/stashNotifier/StashNotifier/config.jelly

    M 
src/main/resources/org/jenkinsci/plugins/stashNotifier/StashNotifier/global.jelly

    M src/test/java/org/jenkinsci/plugins/stashNotifier/ConfigAsCodeTest.java

    M src/test/java/org/jenkinsci/plugins/stashNotifier/StashNotifierTest.java



  Log Message:

  -----------

  Enable InjectedTest & fix found issues



- enable maven-enforcer-plugin to show plugin incompatibilities & fix them

  - without these changes the plugin would fail to start in InjectedTest

  - "git client" plugin v2.0.0 requires "structs" plugin v1.3 or later

    (it was a transitive dependency via "display-url-api" plugin v1.1.1)

  - "git" plugin v3.0.0 requires "credentials" plugin v2.1.4 or later

- all .jelly files should escape content per default

  - this was reported via InjectedTest

  - also see https://wiki.jenkins.io/display/JENKINS/Jelly+and+XSS+prevention

- explicit Hamcrest dependency declaration

  - Hamcrest is used in the tests and therefore should be declared explicitly

  - use the highest version to resolve version conflicts due to transisitivy





  Commit: b924703dd2f100b40f8d4edfbaa6db8804e5b047

      
https://github.com/jenkinsci/stashnotifier-plugin/commit/b924703dd2f100b40f8d4edfbaa6db8804e5b047

  Author: Pavel Batanov <pa...@batanov.me>

  Date:   2020-01-05 (Sun, 05 Jan 2020)



  Changed paths:

    M pom.xml

    M src/main/resources/index.jelly

    M 
src/main/resources/org/jenkinsci/plugins/stashNotifier/StashNotifier/config.jelly

    M 
src/main/resources/org/jenkinsci/plugins/stashNotifier/StashNotifier/global.jelly

    M src/test/java/org/jenkinsci/plugins/stashNotifier/ConfigAsCodeTest.java

    M src/test/java/org/jenkinsci/plugins/stashNotifier/StashNotifierTest.java



  Log Message:

  -----------

  Merge pull request #251 from darxriggs/enable-InjectedTest



Enable InjectedTest & fix found issues





Compare: 
https://github.com/jenkinsci/stashnotifier-plugin/compare/65dbc0080f70...b924703dd2f1

-- 
You received this message because you are subscribed to the Google Groups 
"Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to jenkinsci-commits+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/jenkinsci-commits/jenkinsci/stashnotifier-plugin/push/refs/heads/release/1.x/65dbc0-b92470%40github.com.

Reply via email to