Review comments on earlier merge. (#53221)
This commit is contained in:
parent
d17776af87
commit
51dd95ce35
2 changed files with 4 additions and 3 deletions
|
@ -72,7 +72,7 @@ class AzureDevOpsEntity(Entity):
|
||||||
)
|
)
|
||||||
self._attr_available = False
|
self._attr_available = False
|
||||||
|
|
||||||
async def _azure_devops_update(self) -> None:
|
async def _azure_devops_update(self) -> bool:
|
||||||
"""Update Azure DevOps entity."""
|
"""Update Azure DevOps entity."""
|
||||||
raise NotImplementedError()
|
raise NotImplementedError()
|
||||||
|
|
||||||
|
|
|
@ -97,7 +97,7 @@ class AzureDevOpsLatestBuildSensor(AzureDevOpsSensor):
|
||||||
"mdi:pipe",
|
"mdi:pipe",
|
||||||
)
|
)
|
||||||
|
|
||||||
async def _azure_devops_update(self) -> None:
|
async def _azure_devops_update(self) -> bool:
|
||||||
"""Update Azure DevOps entity."""
|
"""Update Azure DevOps entity."""
|
||||||
try:
|
try:
|
||||||
build: DevOpsBuild = await self.client.get_build(
|
build: DevOpsBuild = await self.client.get_build(
|
||||||
|
@ -106,7 +106,7 @@ class AzureDevOpsLatestBuildSensor(AzureDevOpsSensor):
|
||||||
except aiohttp.ClientError as exception:
|
except aiohttp.ClientError as exception:
|
||||||
_LOGGER.warning(exception)
|
_LOGGER.warning(exception)
|
||||||
self._attr_available = False
|
self._attr_available = False
|
||||||
return
|
return False
|
||||||
self._attr_state = build.build_number
|
self._attr_state = build.build_number
|
||||||
self._attr_extra_state_attributes = {
|
self._attr_extra_state_attributes = {
|
||||||
"definition_id": build.definition.id,
|
"definition_id": build.definition.id,
|
||||||
|
@ -123,3 +123,4 @@ class AzureDevOpsLatestBuildSensor(AzureDevOpsSensor):
|
||||||
"finish_time": build.finish_time,
|
"finish_time": build.finish_time,
|
||||||
}
|
}
|
||||||
self._attr_available = True
|
self._attr_available = True
|
||||||
|
return True
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue