Skip to content

Changing permissions to test build scripts for non ST developers#2869

Merged
aleclorimer merged 1 commit into
mainfrom
jenkins-test-non-st-dev-fix
Apr 3, 2026
Merged

Changing permissions to test build scripts for non ST developers#2869
aleclorimer merged 1 commit into
mainfrom
jenkins-test-non-st-dev-fix

Conversation

@aleclorimer

@aleclorimer aleclorimer commented Apr 1, 2026

Copy link
Copy Markdown
Contributor

Meant to fix an error where non-st developers get a 403 error when trying to trigger the run-jenkins-driver-test job due to permissions errors.

@CLAassistant

CLAassistant commented Apr 1, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@github-actions

github-actions Bot commented Apr 1, 2026

Copy link
Copy Markdown

@github-actions

github-actions Bot commented Apr 1, 2026

Copy link
Copy Markdown

Test Results

   72 files    494 suites   0s ⏱️
2 707 tests 2 707 ✅ 0 💤 0 ❌
4 575 runs  4 575 ✅ 0 💤 0 ❌

Results for commit 72767ab.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Apr 1, 2026

Copy link
Copy Markdown

File Coverage
All files 85%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-sensor/src/lazy_load_subdriver.lua 57%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-sensor/src/init.lua 92%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-sensor/src/sub_drivers/smoke_co_alarm/init.lua 82%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-sensor/src/sensor_utils/device_configuration.lua 89%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-sensor/src/sensor_utils/embedded_cluster_utils.lua 45%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-sensor/src/sub_drivers/air_quality_sensor/air_quality_sensor_utils/utils.lua 74%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-sensor/src/sub_drivers/air_quality_sensor/air_quality_sensor_utils/device_configuration.lua 80%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-sensor/src/sub_drivers/air_quality_sensor/air_quality_sensor_utils/fields.lua 92%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-sensor/src/sub_drivers/air_quality_sensor/air_quality_sensor_utils/legacy_device_configuration.lua 97%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-sensor/src/sub_drivers/air_quality_sensor/init.lua 82%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-sensor/src/sub_drivers/air_quality_sensor/air_quality_sensor_handlers/attribute_handlers.lua 78%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-sensor/src/sub_drivers/bosch_button_contact/init.lua 83%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-sensor/src/sensor_handlers/attribute_handlers.lua 88%

Minimum allowed coverage is 90%

Generated by 🐒 cobertura-action against 72767ab

@aleclorimer aleclorimer force-pushed the jenkins-test-non-st-dev-fix branch 4 times, most recently from 638502f to 837d756 Compare April 2, 2026 19:05
@aleclorimer aleclorimer force-pushed the jenkins-test-non-st-dev-fix branch from 837d756 to 8a4b8f2 Compare April 2, 2026 19:05
@aleclorimer aleclorimer requested a review from kfattig April 2, 2026 19:07
@aleclorimer

Copy link
Copy Markdown
Contributor Author

Reading through documentation online and asking some AI. This should be all that is needed, though its hard to test and verify because I don't have a collaborator account, and could not use a private/personal account as it is un-trusted as a collaborator as well.

@aleclorimer aleclorimer merged commit e763995 into main Apr 3, 2026
1 check passed
@aleclorimer aleclorimer deleted the jenkins-test-non-st-dev-fix branch April 3, 2026 16:40
aleclorimer added a commit that referenced this pull request Apr 28, 2026
…t-dev-fix

Added write permissions for statuses for Jenkins driver test runner job
aleclorimer added a commit that referenced this pull request Apr 29, 2026
…t-dev-fix

Added write permissions for statuses for Jenkins driver test runner job
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants