Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add bun ecosystem FileParser #11521

Merged
merged 1 commit into from
Feb 7, 2025

Conversation

markhallen
Copy link
Contributor

@markhallen markhallen commented Feb 7, 2025

What are you trying to accomplish?

This PR is the next step towards using bun in it's own ecosystem.

Follows #11445, #11446, and #11508

Anything you want to highlight for special attention from reviewers?

We have ignored some opportunities for refactoring as this is mostly moving code from npm_and_yarn to the new javascript parent folder and reorganizing. The new folder structure should allow for future refactoring.

How will you know you've accomplished your goal?

Checklist

  • I have run the complete test suite to ensure all tests and linters pass.
  • I have thoroughly tested my code changes to ensure they work as expected, including adding additional tests for new functionality.
  • I have written clear and descriptive commit messages.
  • I have provided a detailed description of the changes in the pull request, including the problem it addresses, how it fixes the problem, and any relevant details about the implementation.
  • I have ensured that the code is well-documented and easy to understand.

@markhallen markhallen requested a review from a team as a code owner February 7, 2025 15:31
@markhallen markhallen force-pushed the markhallen/add-bun-ecosystem-file-parser branch 3 times, most recently from a489b55 to 7d880ab Compare February 7, 2025 16:17
@robaiken robaiken force-pushed the markhallen/add-bun-ecosystem-file-parser branch from 7d880ab to 9b86e10 Compare February 7, 2025 16:59
@markhallen markhallen merged commit 2d0909d into main Feb 7, 2025
58 checks passed
@markhallen markhallen deleted the markhallen/add-bun-ecosystem-file-parser branch February 7, 2025 18:00
@markhallen markhallen mentioned this pull request Feb 7, 2025
5 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants