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

Rm mshodge from data engineering access #3366

Merged
merged 2 commits into from
Feb 19, 2024
Merged

Rm mshodge from data engineering access #3366

merged 2 commits into from
Feb 19, 2024

Conversation

mshodge
Copy link
Contributor

@mshodge mshodge commented Feb 16, 2024

No description provided.

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Feb 16, 2024 •
edited
Loading

Terraform Component 🧱: github

Checkov 🛂: success

Trivy 🛂: success

Static Analysis Override Label 🏷️: false

Pusher: @calumabarnett, Action: pull_request, Working Directory: terraform/github, Workflow: Terraform, Marker: github_static_analysis

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Feb 16, 2024 •
edited
Loading

Terraform Component 🧱: github

Terraform Initialization ⚙️: success

Terraform Validation 🤖: success

Terraform Plan 🛠️: success

Pusher: @calumabarnett, Action: pull_request, Working Directory: terraform/github, Workflow: Terraform, Marker: github_plan

calumabarnett
calumabarnett previously approved these changes Feb 16, 2024
View reviewed changes
Copy link
Member

@calumabarnett calumabarnett left a comment

Choose a reason for hiding this comment

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

LGTM

@calumabarnett
Copy link
Member

@julialawrence I think you may also need to review

@calumabarnett calumabarnett requested a review from a team February 19, 2024 11:31
jacobwoffenden
jacobwoffenden approved these changes Feb 19, 2024
View reviewed changes
Copy link
Member

@jacobwoffenden jacobwoffenden left a comment

Choose a reason for hiding this comment

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

LGTM

@calumabarnett calumabarnett merged commit 6f28f60 into main Feb 19, 2024
13 checks passed
@calumabarnett calumabarnett deleted the mshodge-patch-1 branch February 19, 2024 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@calumabarnett calumabarnett calumabarnett approved these changes

@jacobwoffenden jacobwoffenden jacobwoffenden approved these changes

@julialawrence julialawrence Awaiting requested review from julialawrence

Assignees

@mshodge mshodge

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mshodge @calumabarnett @jacobwoffenden