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 <[email protected]>

  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 <[email protected]>

  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 [email protected].
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