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 print_github_repository_owners script #4572

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

connormaglynn
Copy link
Contributor

@connormaglynn connormaglynn commented Jun 25, 2024

👀 Purpose

♻️ What's Changed

  • Added a script to map GitHub Repositories to Owners

📝 Notes

  • Not added any unit tests, the main purpose of this is to get the output to discuss with the Lead Engineers, so we may be able to scrap most of this code until we commit to storing these values to feed into a larger system (but at that point we may want a different design)

✅ Things to Check (Optional)

  • I have run all unit tests, and they pass.
  • I have ensured my code follows the project's coding standards.
  • I have checked that all new dependencies are up to date and necessary.

Copy link
Contributor

github-actions bot commented Jun 25, 2024

🦙 MegaLinter status: ❌ ERROR

Descriptor Linter Files Fixed Errors Elapsed time
✅ ACTION actionlint 1 0 0.01s
❌ PYTHON flake8 3 14 2.13s
✅ PYTHON isort 3 1 0 0.59s
❌ PYTHON pylint 3 67 7.62s
✅ REPOSITORY gitleaks yes no 1.67s
✅ YAML prettier 1 0 0 0.39s
✅ YAML yamllint 1 0 0.52s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

@connormaglynn connormaglynn force-pushed the add-print-repo-owners-script branch 23 times, most recently from aafa475 to 0245621 Compare June 25, 2024 17:11
@connormaglynn connormaglynn force-pushed the add-print-repo-owners-script branch 2 times, most recently from ed8a78c to 93046a4 Compare June 26, 2024 16:10
@connormaglynn connormaglynn force-pushed the add-print-repo-owners-script branch 3 times, most recently from 5129c5d to a58377e Compare June 26, 2024 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 👀 Todo
Development

Successfully merging this pull request may close these issues.

1 participant