Skip to content

Add gitleaks pre-commit hook and missing README endpoints#16

Merged
guenp merged 2 commits into
mainfrom
oss-readiness-community-files
Apr 24, 2026
Merged

Add gitleaks pre-commit hook and missing README endpoints#16
guenp merged 2 commits into
mainfrom
oss-readiness-community-files

Conversation

@splch

@splch splch commented Apr 24, 2026

Copy link
Copy Markdown
Collaborator

Summary

Follow-up to #15. Addresses items 6-10 from the open-source readiness audit:

  • Add gitleaks to pre-commit hooks (item 6) - Industry standard secret scanning per OpenSSF Best Practices Badge criteria. Catches secrets before push; GitHub secret scanning (enabled in Add contributor license section and PR template #15) is the backstop.
  • Enable CodeQL default setup (item 7) - Enabled via repo API (Python language). Satisfies OpenSSF Scorecard SAST check.
  • Python version floor (item 8) - Keeping >=3.12 as a deliberate choice. Already documented in README.
  • Add missing endpoints to README table (item 9) - Added get_job_probabilities, get_variant_histogram, get_variant_probabilities, get_variant_shots.
  • Overlay file references (item 10) - No changes needed. The openapi-overlay.yaml is intentionally absent when the upstream spec matches and the existing docs already explain this.

Repo settings updated separately (not in this diff):

Test plan

  • .pre-commit-config.yaml is valid YAML with correct gitleaks rev
  • README endpoint table now lists all modules in ionq_core/api/default/
  • No changes to source code or tests

splch added 2 commits April 24, 2026 09:46
Add gitleaks secret scanning to pre-commit hooks (industry standard
per OpenSSF Best Practices Badge). Add 4 missing endpoint modules to
the README's available endpoints table: get_job_probabilities,
get_variant_histogram, get_variant_probabilities, get_variant_shots.
@guenp guenp merged commit c98809a into main Apr 24, 2026
9 checks passed
@guenp guenp deleted the oss-readiness-community-files branch April 24, 2026 19:39
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