chore: replace `pnpm run` with `concurrently` to maintain cross-platform tolerance
Issue
- resolve: https://github.com/liam-hq/liam/issues/1212
Why is this change needed?
What would you like reviewers to focus on?
Testing Verification
What was done
🤖 Generated by PR Agent at d5696a05cdb7b6dc1ac038570259e5a2f2f04d9d
- Replaced
pnpm runwithconcurrentlyfor cross-platform compatibility. - Updated all
package.jsonscripts to useconcurrentlyfor pattern matching. - Added
concurrentlyas a new dependency in the rootpackage.json. - Updated
pnpm-lock.yamlto includeconcurrentlyand its dependencies.
Detailed Changes
| Relevant files | |||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Enhancement | 12 files
| ||||||||||||||||||||||||
| Dependencies |
Additional Notes
I do modified pnpm-lock.yaml with only concurrently and it's dependency without modifying other package's version, I believe this will not change the functionality of other packages.
Need help?
Type /help how to ...in the comments thread for any questions about Qodo Merge usage.Check out the documentation for more information.
⚠️ No Changeset found
Latest commit: d2a2a09e798b5851fd4decd396f8920cea7376f9
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
@samuel871211 is attempting to deploy a commit to the ROUTE06 Core Team on Vercel.
A member of the Team first needs to authorize it.
PR Reviewer Guide 🔍
Here are some key observations to aid the review process:
|
🎫 Ticket compliance analysis ✅ 1212 - Fully compliant Compliant requirements: • Fix the issue where |
| ⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪ |
| 🧪 No relevant tests |
| 🔒 No security concerns identified |
| ⚡ No major issues detected |
PR Code Suggestions ✨
Explore these optional code suggestions:
| Category | Suggestion | Impact |
| Possible issue |
✅
| Medium |
| ||
https://github.com/liam-hq/liam/pull/1216#issuecomment-2783330707
I'm not sure whether to run the changeset or not, if it;s required, please run the changeset for me, thank you.
Hi @hoshinotsuyoshi I am not sure if this error is related to this PR Could you take a look at this error? I think it's related to this commit
The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| liam-app | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Apr 8, 2025 0:25am |
| liam-erd-sample | ✅ Ready (Inspect) | Visit Preview | Apr 8, 2025 0:25am |
1 Skipped Deployment
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| liam-docs | ⬜️ Ignored (Inspect) | Visit Preview | Apr 8, 2025 0:25am |
@samuel871211 That's the problem that happens when it's a PR from a fork. Thanks, you can ignore it for now.