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

Update from template 2024-08 #4

Merged
merged 6 commits into from
Sep 19, 2024
Merged

Update from template 2024-08 #4

merged 6 commits into from
Sep 19, 2024

Conversation

doubleedesign
Copy link

Summary

What is the nature of this pull request?

  • New feature
  • Enhancement to an existing feature
  • Bug fix
  • Configuration update
  • Dependency update

Description of change

Updating this repo with upstream configuration changes.

Planner card link

N/A

Readiness

  • The application runs successfully with my change.
  • N/A I have included code comments where appropriate.
  • N/A Where I have copied and pasted code from other sources, I have included a comment with a link to the original source.
  • I have read and followed the principles and guidelines about submitting code.
  • N/A I have added unit tests for new components and/or functionality.
  • All existing and new unit tests pass (including updating tests for any necessary breaking changes).

@doubleedesign doubleedesign requested review from a team as code owners August 25, 2024 06:18
Ashan224019868
Ashan224019868 approved these changes Sep 19, 2024
View reviewed changes
Copy link

@Ashan224019868 Ashan224019868 left a comment

Choose a reason for hiding this comment

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

lgtm-No Security Issues

ben-AI-cybersec
ben-AI-cybersec approved these changes Sep 19, 2024
View reviewed changes
Copy link
Member

@ben-AI-cybersec ben-AI-cybersec left a comment

Choose a reason for hiding this comment

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

Linting changes, approved

@ben-AI-cybersec ben-AI-cybersec merged commit ff30315 into main Sep 19, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Ashan224019868 Ashan224019868 Ashan224019868 approved these changes

@ben-AI-cybersec ben-AI-cybersec ben-AI-cybersec approved these changes

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

Successfully merging this pull request may close these issues.
3 participants
@doubleedesign @ben-AI-cybersec @Ashan224019868