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.

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

Add back basic repo files #23

Closed
wants to merge 1 commit into from
Closed

Conversation

matthewkotila
Copy link
Contributor

@matthewkotila matthewkotila commented Aug 1, 2024

This adds back

  • .clang-format
  • LICENSE
  • .pre-commit-config.yaml
  • .github/workflows/codeql.yaml
  • .github/workflows/pre-commit.yaml
  • .github/workflows/python-package-genai.yaml

@dyastremsky
Copy link
Contributor

dyastremsky commented Aug 5, 2024

Great minds think alike! I have the automation files in this PR: #20. Pre-commit is also passing in that PR now. I'm still awaiting this repo to be made public and/or security scanning to be enabled to merge it?

Clang-format should be good to remove now, since we use the clang-format pre-commit hook instead. It looks like you added it in #24 as well.

@fpetrini15 has a PR open for the license here: #21

We could consolidate PRs so that we don't have duplication. We could use whichever approach makes sense.

@matthewkotila
Copy link
Contributor Author

@dyastremsky I'll close this--I made it after realizing other PRs existed doing the same.

@matthewkotila matthewkotila deleted the mkotila-repo-files branch August 7, 2024 01:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants