Skip to content

Comments

PR attestation enhancement#514

Merged
sami-alajrami merged 20 commits intomainfrom
pr-attestation-enhaancement
Jul 4, 2025
Merged

PR attestation enhancement#514
sami-alajrami merged 20 commits intomainfrom
pr-attestation-enhaancement

Conversation

@sami-alajrami
Copy link
Contributor

use Github's GraphQL API to capture more details about github pull requests.

@sami-alajrami sami-alajrami marked this pull request as draft July 3, 2025 12:18
@sami-alajrami sami-alajrami marked this pull request as ready for review July 3, 2025 13:29
@sami-alajrami sami-alajrami enabled auto-merge (squash) July 4, 2025 10:24
@sami-alajrami sami-alajrami merged commit 9b4efda into main Jul 4, 2025
9 checks passed
@sami-alajrami sami-alajrami deleted the pr-attestation-enhaancement branch July 4, 2025 11:18
github-actions bot pushed a commit that referenced this pull request Jul 11, 2025
* Implement optional flag and loop to poll SonarQube and wait for scan to complete

* Add missing argument to NewSonarConfig

* Add 'IN_PROGRESS' status to sleep loop

* Add more thorough error message to attest sonar command

* Change --allow-wait flag to take number of seconds to wait as parameter

* Clean up leftover comment

* Add more details on new flag to attest sonar description

* Change --allow-wait flag to --max-retries

* Change --allow-wait flag in example commands for docs

* Fix typo

* Start implementing GraphQL for Github PRs

* Update variable names for PR structs to match API

* Update PR struct to work with both old and new versions of PR attestation

* Fix payload for non-github PR attestations and tidy up comments

* Fix payload issues for non-Github-attest commands

* Update report evidence command payloads

* Get git provider for reporting commit evidence properly

* add author_username to github pr attestation commits

---------

Co-authored-by: Faye <[email protected]>
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.

2 participants