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

B 18490 update mto v2 final destination sit #11913

Merged
merged 7 commits into from
Feb 20, 2024
Merged

B 18490 update mto v2 final destination sit #11913

merged 7 commits into from
Feb 20, 2024

Conversation

cameroncaci
Copy link
Contributor

@cameroncaci cameroncaci commented Feb 2, 2024 •
edited
Loading

@cameroncaci cameroncaci self-assigned this Feb 5, 2024
@mergify mergify bot mentioned this pull request Feb 5, 2024
Merged
@cameroncaci cameroncaci added the Mountain Movers Movin' Mountains 1 Sprint at a time label Feb 5, 2024
@cameroncaci cameroncaci marked this pull request as ready for review February 20, 2024 19:32
@cameroncaci cameroncaci requested a review from a team as a code owner February 20, 2024 19:32
@danieljordan-caci
Copy link
Contributor

Any known reason for the discrepancies in code changes between your integrationTesting branch and this one?

@traskowskycaci
Copy link
Contributor

Any known reason for the discrepancies in code changes between your integrationTesting branch and this one?

Probably that homesafe prd ssl commit that likely came through on grabbing the latest from main and getting that into integration. I see the same commits + commit hashes for Cameron's work so I am approving

@danieljordan-caci
Copy link
Contributor

Any known reason for the discrepancies in code changes between your integrationTesting branch and this one?

Probably that homesafe prd ssl commit that likely came through on grabbing the latest from main and getting that into integration. I see the same commits + commit hashes for Cameron's work so I am approving

Noice catch

danieljordan-caci
danieljordan-caci approved these changes Feb 20, 2024
View reviewed changes
Copy link
Contributor

@danieljordan-caci danieljordan-caci left a comment

Choose a reason for hiding this comment

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

You know what they say... "Looks good to me baybay"

@deandreJones deandreJones merged commit 7fd5ae0 into main Feb 20, 2024
38 checks passed
@deandreJones deandreJones deleted the B-18490-UpdateMTO-V2-FinalDestinationSIT branch February 20, 2024 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@deandreJones deandreJones deandreJones approved these changes

@danieljordan-caci danieljordan-caci danieljordan-caci approved these changes

@traskowskycaci traskowskycaci traskowskycaci approved these changes

@TevinAdams TevinAdams Awaiting requested review from TevinAdams

Assignees

@cameroncaci cameroncaci

Labels
Mountain Movers Movin' Mountains 1 Sprint at a time
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@cameroncaci @danieljordan-caci @traskowskycaci @deandreJones