Skip to content

chore: fix lint-staged failed when js/ts file staged#9246

Merged
JSerFeng merged 6 commits intomainfrom
chore/fix-lint-staged
Feb 12, 2025
Merged

chore: fix lint-staged failed when js/ts file staged#9246
JSerFeng merged 6 commits intomainfrom
chore/fix-lint-staged

Conversation

@stormslowly
Copy link
Copy Markdown
Contributor

@stormslowly stormslowly commented Feb 11, 2025

Summary

  • format:js format only staged files; No other glob files
  • move lint staged to js config file, for better config format

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

No more -n when commit

@netlify
Copy link
Copy Markdown

netlify bot commented Feb 11, 2025

Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit 64a1a13
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/67ac1f295dddfc00080d91ee

SoonIter
SoonIter previously approved these changes Feb 11, 2025
@SoonIter SoonIter dismissed their stale review February 11, 2025 11:29

🚫💩

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Feb 11, 2025

CodSpeed Performance Report

Merging #9246 will not alter performance

Comparing chore/fix-lint-staged (64a1a13) with main (7820021)

🎉 Hooray! codspeed-node just leveled up to 4.0.0!

A heads-up, this is a breaking change and it might affect your current performance baseline a bit. But here's the exciting part - it's packed with new, cool features and promises improved result stability 🥳!
Curious about what's new? Visit our releases page to delve into all the awesome details about this new version.

Summary

✅ 6 untouched benchmarks

@stormslowly stormslowly force-pushed the chore/fix-lint-staged branch from 58a1ed0 to 5d9f52e Compare February 11, 2025 16:29
@stormslowly stormslowly requested a review from JSerFeng February 12, 2025 04:05
@JSerFeng JSerFeng merged commit 1875e86 into main Feb 12, 2025
33 checks passed
@JSerFeng JSerFeng deleted the chore/fix-lint-staged branch February 12, 2025 06:09
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