Skip to content

docs: Refine OIDC docs - #1837

Merged
spalmurray merged 2 commits into
mainfrom
spalmurray/oidc-docs
Jun 3, 2025
Merged

docs: Refine OIDC docs#1837
spalmurray merged 2 commits into
mainfrom
spalmurray/oidc-docs

Conversation

@spalmurray

@spalmurray spalmurray commented Jun 3, 2025

Copy link
Copy Markdown
Contributor

Updates the OIDC docs to mention the need for the id-token: write permission. Also my formatter decided README was ugly

spalmurray
spalmurray commented Jun 3, 2025
View reviewed changes
Comment thread README.md

### Using OIDC
For users with [OpenID Connect(OIDC) enabled](https://docs.github.com/en/actions/deployment/security-hardening-your-deployments/about-security-hardening-with-openid-connect), the Codecov token is not necessary. You can use OIDC with the `use_oidc` argument as following.

@spalmurray spalmurray Jun 3, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is the real change. Up to line 123

@spalmurray
spalmurray marked this pull request as ready for review June 3, 2025 15:31
@spalmurray
spalmurray merged commit 5ecdce8 into main Jun 3, 2025
@spalmurray
spalmurray deleted the spalmurray/oidc-docs branch June 3, 2025 21:03
@codecov

codecov Bot commented Jun 3, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 0.00%. Comparing base (15559ed) to head (437034d).
Report is 1 commits behind head on main.

Additional details and impacted files 
@@     Coverage Diff      @@
##   main   #1837   +/-   ##
============================
============================

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

mergify Bot added a commit to ArcadeData/arcadedb that referenced this pull request Mar 23, 2026
@mergify