Skip to content

chore: fix single lint warning#516

Merged
43081j merged 1 commit intobombshell-dev:mainfrom
ghostdevv:lint
Apr 23, 2026
Merged

chore: fix single lint warning#516
43081j merged 1 commit intobombshell-dev:mainfrom
ghostdevv:lint

Conversation

@ghostdevv
Copy link
Copy Markdown
Contributor

What does this PR do?

Just fixes a single lint error, but I don't know if I have a better PR to sneak this in with 😄

Type of change

  • Bug fix
  • Feature
  • Refactor (no behavior change)
  • Documentation
  • Performance improvement
  • Tests
  • Chore (dependencies, CI, tooling)

Checklist

  • pnpm test passes (or targeted tests for my change)
  • pnpm format has been run
  • I have added/updated tests for my changes (if applicable)
  • I have added a changeset

AI-generated code disclosure

  • This PR includes AI-generated code

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 21, 2026

⚠️ No Changeset found

Latest commit: 55dee7d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@ghostdevv ghostdevv changed the title chore: fix lint error chore: fix single lint warning Apr 21, 2026
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Apr 22, 2026

commit: 55dee7d

@43081j
Copy link
Copy Markdown
Collaborator

43081j commented Apr 22, 2026

how did you encounter this warning?

pnpm lint seems to pass for me with no warnings

@ghostdevv
Copy link
Copy Markdown
Contributor Author

how did you encounter this warning?

pnpm lint seems to pass for me with no warnings

Just running pnpm lint - I'm running it from the root of the monorepo too if that makes a difference

@43081j
Copy link
Copy Markdown
Collaborator

43081j commented Apr 23, 2026

me too and no warnings.

what exactly is the warning/error you get? can you show the actual message?

@gameroman
Copy link
Copy Markdown

image image

@43081j
Copy link
Copy Markdown
Collaborator

43081j commented Apr 23, 2026

after nuking my local copy i can see it too 👍

no clue what made biome miss that

@43081j 43081j merged commit c51243b into bombshell-dev:main Apr 23, 2026
9 checks passed
@ghostdevv ghostdevv deleted the lint branch April 24, 2026 02:23
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.

3 participants