tree 2a29e263d1ca2e488896e4aa3fce73bab51c702d
parent 6d7a0e84cee81fc1ed9c8d789eb5465c893fe85f
parent 7b6ad0e8819e93d3e4897dbeae5df65209cb86f4
author David Ostrovsky <david@ostrovsky.org> 1529301862 +0200
committer David Ostrovsky <david@ostrovsky.org> 1529388581 +0200

Merge branch 'stable-2.15'

* stable-2.15: (43 commits)
  Simplify client module
  Split reviewers plugin into GWT and non-GWT variants
  Update bazlets to latest revision on stable-2.15
  Add modify reviewers config plugin capability
  Fix classpath generation for Eclipse project
  Update bazlets to latest revision on stable-2.15
  ReviewersResolver: Log failure to list accounts for group as error
  ReviewersResolver: Don't log stack trace when group does not exist
  ReviewersResolver: Emit a warning to the log when account is inactive
  Update bazlets to latest stable-2.15 to use 2.15.1 API
  Update bazlets to latest stable-2.14 and switch to 2.14.8 release API
  Fix compilation warnings flagged by Eclipse
  Upgrade bazlets and use 2.15 release API
  Reviewers: Restrict index query to the given change
  Build with 2.15-SNAPSHOT API
  Update bazlets to latest on stable-2.15
  Add integration test for adding reviewers
  ReviewersConfig: Add unit test for merged project inheritance
  ReviewerFilterSection: Add equals and hashCode methods
  ReviewersResolver#resolveGroup: Simplify error handling
  Remove stale documentation how to build with maven
  Add reviewers resolver test
  ReviewersResolver#resolve: Extract logic in small methods
  Reviewers: Extract account resolver logic to it own class
  Add members of all matching groups
  ReviewersConfig: merge parent reviewers.config
  Update bazlets to latest stable-2.14 to use 2.14.8-SNAPSHOT API
  Update bazlets to the latest on stable-2.14 to use 2.14.7 release API
  Reviewers: Don't create ChangeData instances
  Reviewers: Actually use change index for filter query
  Refactor ReviewersConfig to also handle the global config
  Rename DefaultReviewers to AddReviewersByConfiguration
  Factor addition of reviewers into own Runnable class
  Reviewers: Don't add change uploader as reviewer
  Reviewers: Only consider active accounts
  Reviewers: Pass RevisionEvent to onEvent method
  Add project name & change Id to the logs
  Reviewers: Use log built-in string formatting
  Handle the QueryParseException when querying a group that does not exist
  Reviewers#suggestReviewers: Add null check on changeId
  Add support for reviewer suggestion
  ChangeEventListener: Remove unnecessary final modifiers
  ChangeEventListener: Don't unnecessarily open the repository

Change-Id: I74e11c3f770ecf923b78a1e792a59365e8a062a5