Skip to content

[FIX] Correct typo which led to invalid credential#259

Merged
PGijsbers merged 1 commit intomainfrom
PGijsbers-patch-1
Feb 26, 2026
Merged

[FIX] Correct typo which led to invalid credential#259
PGijsbers merged 1 commit intomainfrom
PGijsbers-patch-1

Conversation

@PGijsbers
Copy link
Copy Markdown
Contributor

DOCKER_PASSWORD is not a configured secret for this repository.

`DOCKER_PASSWORD` is not a configured secret for this repository.
@PGijsbers PGijsbers added bug Something isn't working dev-tools Anything related to the developer tooling (CI, pre-commit, etc.) automation CI/CD, pre-commit, ... labels Feb 26, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Feb 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6a0fed5 and 739a21d.

📒 Files selected for processing (1)
  • .github/workflows/docker-publish.yml

Walkthrough

The pull request modifies the Docker publish workflow configuration by updating the Docker Hub description step. The change updates the password credential reference from DOCKERHUB_PASSWORD to DOCKERHUB_TOKEN in the with.password input parameter. This is a single-line modification that changes which environment variable is used for authentication in this step. No changes were made to other workflow logic, error handling, or exported declarations.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: fixing a typo related to invalid credentials, which aligns with the changeset updating the Docker Hub password credential reference.
Description check ✅ Passed The description is directly related to the changeset, explaining that DOCKER_PASSWORD is not a configured secret and the PR fixes this issue.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch PGijsbers-patch-1

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

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

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@PGijsbers PGijsbers merged commit a887aa1 into main Feb 26, 2026
3 checks passed
@PGijsbers PGijsbers deleted the PGijsbers-patch-1 branch February 26, 2026 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automation CI/CD, pre-commit, ... bug Something isn't working dev-tools Anything related to the developer tooling (CI, pre-commit, etc.)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant