Branch: refs/heads/master
  Home:   https://github.com/jenkinsci/stash-pullrequest-builder-plugin
  Commit: d2c6a4b8177c8ce4dc6a53222ae2248ca850d04d
      
https://github.com/jenkinsci/stash-pullrequest-builder-plugin/commit/d2c6a4b8177c8ce4dc6a53222ae2248ca850d04d
  Author: Pavel Roskin <plros...@gmail.com>
  Date:   2019-11-25 (Mon, 25 Nov 2019)

  Changed paths:
    M 
src/main/java/stashpullrequestbuilder/stashpullrequestbuilder/StashBuildTrigger.java
    M 
src/main/java/stashpullrequestbuilder/stashpullrequestbuilder/StashRepository.java
    M 
src/test/java/stashpullrequestbuilder/stashpullrequestbuilder/StashRepositoryTest.java

  Log Message:
  -----------
  StashRepository: Take pollLog as a constructor argument

Move the code that logs common information (start and stop time,
duration, number of pull requests) to pollRepository(). Add a unit test
to check that the common information is being logged.


  Commit: 9d21d4f7a99628c21002b6a4db55ab6586ddff24
      
https://github.com/jenkinsci/stash-pullrequest-builder-plugin/commit/9d21d4f7a99628c21002b6a4db55ab6586ddff24
  Author: Pavel Roskin <plros...@gmail.com>
  Date:   2019-11-25 (Mon, 25 Nov 2019)

  Changed paths:
    M 
src/main/java/stashpullrequestbuilder/stashpullrequestbuilder/StashRepository.java

  Log Message:
  -----------
  StashRepository: Log exceptions to the poll log and to the system log

Even users without administrative access should know if their PRs are not
built due to an exception.


  Commit: c8f0efb629c17307ff440707b3a0f27d1a46fa3d
      
https://github.com/jenkinsci/stash-pullrequest-builder-plugin/commit/c8f0efb629c17307ff440707b3a0f27d1a46fa3d
  Author: Pavel Roskin <plros...@gmail.com>
  Date:   2019-11-25 (Mon, 25 Nov 2019)

  Changed paths:
    M 
src/main/java/stashpullrequestbuilder/stashpullrequestbuilder/StashRepository.java

  Log Message:
  -----------
  StashRepository: Log triggering decisions to the poll log only

If a message explains why a PR is not being built, log it to the poll log
only, unless it's caused by an exception.

Triggering decisions depend on the state of the PR and don't appear and
disappear on their own. It is not of much interest why a PR wasn't built
in the past.

No need to fill logs with messages about pull requests that cannot be
merged for every polling cycle.


Compare: 
https://github.com/jenkinsci/stash-pullrequest-builder-plugin/compare/70be4cf1df7e...c8f0efb629c1

-- 
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/stash-pullrequest-builder-plugin/push/refs/heads/master/70be4c-c8f0ef%40github.com.

Reply via email to