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

fix: Remove unused token variable #10302

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kasperbirch1
Copy link

What?

This pull request removes an unused token variable, left over from commit d8c106c#diff-21d57510d3adf8f27cc3755ee834a1dbfb60fe1beb53f47abef824b130d8f278

Why?

I really appreciate Payload CMS and wanted to contribute by cleaning up minor technical debt. This is my first pull request, so please let me know if there’s anything else I should add or revise.

How?

  • Removed the token variable
  • Verified it was not referenced elsewhere, thus no additional refactoring was required.

Copy link
Contributor

github-actions bot commented Jan 2, 2025

Pull Request titles must follow the Conventional Commits specification and have valid scopes.

The subject "Remove unused token variable" found in the pull request title "fix: Remove unused token variable"
didn't match the configured pattern. Please ensure that the subject
doesn't start with an uppercase character.

feat(ui): add Button component
^    ^    ^
|    |    |__ Subject
|    |_______ Scope
|____________ Type

@kasperbirch1 kasperbirch1 changed the title Fix: Remove unused token variable fix: Remove unused token variable Jan 2, 2025
The `token` variable is not in use and is a leftover from commit `d8c106cb2b8fa157d2ea545991abd1d851a733ec`.
@kasperbirch1 kasperbirch1 force-pushed the remove-unused-token-variable branch from e8a5118 to eaa2b56 Compare January 2, 2025 14:05
@jacobsfletch jacobsfletch added examples Examples directory area: templates Templates directory labels Jan 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: templates Templates directory examples Examples directory
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants