Add capability for global laziness (PyGithub#2746)

Adds capability to configure global laziness Github objects through
`Github(lazy=…)` and `GithubIntegration(lazy=…)`. Does not add any
laziness capability to any particular Github object.

Reworks `CompletableGithubObject` to support following features:
- It fetches initial complete JSON, which allows to replace
```python
headers, data = self.__requester.requestJsonAndCheck("GET", f"/users/{login}")
return github.NamedUser.NamedUser(self.__requester, headers, data, completed=True)
```
with
```python
return github.NamedUser.NamedUser(self.__requester, url=f"/users/{login}")
```
Which allows that returned `CompletableGithubObject` to be lazy.

Removes any code related to `CompletableGithubObject` from
`NonCompletableGithubObject`.
- Which requires replacing (for `NonCompletableGithubObject` only)
```python
        return github.Branch.Branch(self._requester, headers, data, completed=True)
```
for `NonCompletableGithubObject` with
```python
        return github.Branch.Branch(self._requester, headers, data)
```

Fixes PyGithub#2334

Author: EnricoMi

github/ApplicationOAuth.py

@@ -62,11 +62,10 @@ def __init__(
         requester: Requester,
         headers: dict[str, Any],
         attributes: Any,
-        completed: bool,
     ) -> None:
         # this object requires a request without authentication
         requester = requester.withAuth(auth=None)
-        super().__init__(requester, headers, attributes, completed)
+        super().__init__(requester, headers, attributes)
 
     def __repr__(self) -> str:
         return self.get__repr__({"client_id": self._client_id.value})
@@ -146,7 +145,6 @@ def get_access_token(self, code: str, state: str | None = None) -> AccessToken:
             requester=self._requester,
             headers=headers,
             attributes=data,
-            completed=False,
         )
 
     def get_app_user_auth(self, token: AccessToken) -> AppUserAuth:
@@ -187,7 +185,6 @@ def refresh_access_token(self, refresh_token: str) -> AccessToken:
             requester=self._requester,
             headers=headers,
             attributes=data,
-            completed=False,
         )
 
     @staticmethod

github/Auth.py

@@ -451,7 +451,6 @@ def withRequester(self, requester: Requester) -> "AppUserAuth":
                 "client_id": self._client_id,
                 "client_secret": self._client_secret,
             },
-            completed=False,
         )
 
         return self

github/Commit.py

@@ -204,7 +204,7 @@ def create_status(
             f"{self._parentUrl(self._parentUrl(self.url))}/statuses/{self.sha}",
             input=post_parameters,
         )
-        return github.CommitStatus.CommitStatus(self._requester, headers, data, completed=True)
+        return github.CommitStatus.CommitStatus(self._requester, headers, data)
 
     def get_comments(self) -> PaginatedList[CommitComment]:
         """
@@ -233,7 +233,7 @@ def get_combined_status(self) -> CommitCombinedStatus:
         :calls: `GET /repos/{owner}/{repo}/commits/{ref}/status/ <http://docs.github.com/en/rest/reference/repos#statuses>`_
         """
         headers, data = self._requester.requestJsonAndCheck("GET", f"{self.url}/status")
-        return github.CommitCombinedStatus.CommitCombinedStatus(self._requester, headers, data, completed=True)
+        return github.CommitCombinedStatus.CommitCombinedStatus(self._requester, headers, data)
 
     def get_pulls(self) -> PaginatedList[PullRequest]:
         """

github/Enterprise.py

@@ -61,7 +61,7 @@ def __init__(
         enterprise: str,
     ):
         enterprise = urllib.parse.quote(enterprise)
-        super().__init__(requester, {}, {"enterprise": enterprise, "url": f"/enterprises/{enterprise}"}, True)
+        super().__init__(requester, {}, {"enterprise": enterprise, "url": f"/enterprises/{enterprise}"})
 
     def _initAttributes(self) -> None:
         self._enterprise: Attribute[str] = NotSet

github/GithubIntegration.py

@@ -83,6 +83,8 @@ def __init__(
         jwt_issued_at: int = Consts.DEFAULT_JWT_ISSUED_AT,
         jwt_algorithm: str = Consts.DEFAULT_JWT_ALGORITHM,
         auth: AppAuth | None = None,
+        # v3: set lazy = True as the default
+        lazy: bool = False,
     ) -> None:
         """
         :param integration_id: int deprecated, use auth=github.Auth.AppAuth(...) instead
@@ -100,6 +102,8 @@ def __init__(
         :param jwt_issued_at: int deprecated, use auth=github.Auth.AppAuth(...) instead
         :param jwt_algorithm: string deprecated, use auth=github.Auth.AppAuth(...) instead
         :param auth: authentication method
+        :param lazy: completable objects created from this instance are lazy,
+                     as well as completable objects created from those, and so on
         """
         if integration_id is not None:
             assert isinstance(integration_id, (int, str)), integration_id
@@ -117,6 +121,7 @@ def __init__(
         assert isinstance(jwt_expiry, int), jwt_expiry
         assert Consts.MIN_JWT_EXPIRY <= jwt_expiry <= Consts.MAX_JWT_EXPIRY, jwt_expiry
         assert isinstance(jwt_issued_at, int)
+        assert isinstance(lazy, bool), lazy
 
         self.base_url = base_url
 
@@ -157,8 +162,22 @@ def __init__(
             pool_size=pool_size,
             seconds_between_requests=seconds_between_requests,
             seconds_between_writes=seconds_between_writes,
+            lazy=lazy,
         )
 
+    def withLazy(self, lazy: bool) -> GithubIntegration:
+        """
+        Create a GithubIntegration instance with identical configuration but the given lazy setting.
+
+        :param lazy: completable objects created from this instance are lazy, as well as completable objects created
+            from those, and so on
+        :return: new Github instance
+
+        """
+        kwargs = self.__requester.kwargs
+        kwargs.update(lazy=lazy)
+        return GithubIntegration(**kwargs)
+
     def close(self) -> None:
         """Close connections to the server. Alternatively, use the
         GithubIntegration object as a context manager:
@@ -211,7 +230,6 @@ def _get_installed_app(self, url: str) -> Installation:
             requester=self.__requester,
             headers=headers,
             attributes=response,
-            completed=True,
         )
 
     @deprecated.deprecated(
@@ -248,7 +266,6 @@ def get_access_token(
             requester=self.__requester,
             headers=headers,
             attributes=response,
-            completed=True,
         )
 
     @deprecated.deprecated("Use get_repo_installation")