Iterator.remove() has a default implementation

src/main/java/org/kohsuke/github/GHNotificationStream.java

@@ -207,10 +207,6 @@ private long calcNextCheckTime(GitHubResponse<GHThread[]> response) {
                 long seconds = Integer.parseInt(v);
                 return System.currentTimeMillis() + seconds * 1000;
             }
-
-            public void remove() {
-                throw new UnsupportedOperationException();
-            }
         };
     }
 

src/main/java/org/kohsuke/github/GHPerson.java

@@ -131,10 +131,6 @@ public List<GHRepository> next() {
                             r.root = root;
                         return Arrays.asList(batch);
                     }
-
-                    public void remove() {
-                        throw new UnsupportedOperationException();
-                    }
                 };
             }
         };

src/main/java/org/kohsuke/github/GitHubPageIterator.java

@@ -114,10 +114,6 @@ public GitHubResponse<T> finalResponse() {
         return finalResponse;
     }
 
-    public void remove() {
-        throw new UnsupportedOperationException();
-    }
-
     /**
      * Fetch is called at the start of {@link #hasNext()} or {@link #next()} to fetch another page of data if it is
      * needed.

src/main/java/org/kohsuke/github/PagedIterator.java

@@ -97,10 +97,6 @@ private void fetch() {
         }
     }
 
-    public void remove() {
-        throw new UnsupportedOperationException();
-    }
-
     /**
      * Gets the next page worth of data.
      *

src/main/java/org/kohsuke/github/PagedSearchIterable.java

@@ -57,11 +57,11 @@ private void populate() {
     }
 
     /**
-     * Adapts {@link Iterator}.
+     * Adapts from {@link Iterator<SearchResult<T>>} to {@link Iterator<T[]>}.
      *
      * @param base
-     *            the base
-     * @return the iterator
+     *            {@link Iterator<SearchResult<T>>} to adapt.
+     * @return a {@link Iterator<T[]>} wrapping the base {@link Iterator<SearchResult<T>>}.
      */
     protected Iterator<T[]> adapt(final Iterator<? extends SearchResult<T>> base) {
         return new Iterator<T[]>() {
@@ -75,10 +75,6 @@ public T[] next() {
                     result = v;
                 return v.getItems(root);
             }
-
-            public void remove() {
-                throw new UnsupportedOperationException();
-            }
         };
     }
 }

src/test/java/org/kohsuke/github/GitHubTest.java

@@ -53,6 +53,14 @@ public void testListAllRepositories() throws Exception {
             assertNotNull(r.getUrl());
             assertNotEquals(0L, r.getId());
         }
+
+        // ensure the iterator throws as expected
+        try {
+            itr.remove();
+            fail();
+        } catch (UnsupportedOperationException e) {
+            assertThat(e, notNullValue());
+        }
     }
 
     @Test