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

Adding a warning for the user when the SCM project doesn't match (AST-78507) #297

Merged
merged 4 commits into from
Jan 2, 2025
Merged

Adding a warning for the user when the SCM project doesn't match (AST-78507) #297

merged 4 commits into from
Jan 2, 2025

Conversation

elchananarb
Copy link
Contributor

Description

Adding a warning for the user when the SCM project doesn't match the CX project.

References

https://checkmarx.atlassian.net/browse/AST-78507

Testing

Describe how this change was tested. Be specific about anything not tested and reasons why. If this solution has unit and/or integration testing, tests should be added for new functionality and existing tests should complete without errors.

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).
  • All active GitHub checks for tests, formatting, and security are passing
  • The correct base branch is being used

@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 26, 2024 •
edited
Loading

Logo
Checkmarx One – Scan Summary & Details – e47e2b64-60f4-4468-9919-bfaf00463f66

No New Or Fixed Issues Found

AlvoBen
AlvoBen requested changes Dec 31, 2024
View reviewed changes
Copy link
Contributor

@AlvoBen AlvoBen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fix test

@elchananarb elchananarb merged commit 896b74f into main Jan 2, 2025
6 checks passed
@elchananarb elchananarb deleted the feature/elchanan/adding_warning_project_doesnt_match branch January 2, 2025 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@miryamfoiferCX miryamfoiferCX miryamfoiferCX left review comments

@AlvoBen AlvoBen AlvoBen approved these changes

@greensd4 greensd4 greensd4 approved these changes

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

Successfully merging this pull request may close these issues.
4 participants
@elchananarb @greensd4 @AlvoBen @miryamfoiferCX