Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add Pr-Decoration Support On ADO (AST-70135) #942

Merged
merged 12 commits into from
Nov 14, 2024
Merged

Add Pr-Decoration Support On ADO (AST-70135) #942

merged 12 commits into from
Nov 14, 2024

Conversation

miryamfoiferCX
Copy link
Contributor

By submitting a PR to this repository, you agree to the terms within the Checkmarx Code of Conduct. Please see the contributing guidelines for how to create and submit a high-quality PR for this repo.

Description

Add Pr-Decoration Support on ADO

References

Include supporting link to GitHub Issue/PR number

Testing

integration test
unit test
E2E test

Please include any manual steps for testing end-to-end or functionality not covered by unit/integration tests.

Checklist

  • I have added documentation for new/changed functionality in this PR (if applicable).
  • I have updated the CLI help for new/changed functionality in this PR (if applicable).
  • All active GitHub checks for tests, formatting, and security are passing
  • The correct base branch is being used

@OrShamirCM OrShamirCM requested a review from a team November 13, 2024 13:45
@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 13, 2024 •
edited
Loading

Logo
Checkmarx One – Scan Summary & Details – 5fa8bc57-ad67-42d7-9a26-5b92b327e255

No New Or Fixed Issues Found

@miryamfoiferCX miryamfoiferCX marked this pull request as ready for review November 14, 2024 07:46
AlvoBen
AlvoBen reviewed Nov 14, 2024
View reviewed changes
internal/commands/util/pr.go Outdated Show resolved Hide resolved
internal/commands/util/pr.go Show resolved Hide resolved
internal/commands/util/pr.go Outdated Show resolved Hide resolved
internal/commands/util/pr_test.go Show resolved Hide resolved
@miryamfoiferCX miryamfoiferCX merged commit e55d4cc into main Nov 14, 2024
9 checks passed
@miryamfoiferCX miryamfoiferCX deleted the feature/MiryamFoifer/PRDecorationSupportOnADO branch November 14, 2024 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AlvoBen AlvoBen AlvoBen approved these changes

@OrShamirCM OrShamirCM Awaiting requested review from OrShamirCM OrShamirCM is a code owner

Assignees
No one assigned
Labels
feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@miryamfoiferCX @AlvoBen