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

chore: Move prettier.config.js settings into package.json #27

Merged
merged 1 commit into from
Sep 19, 2024

Conversation

markstur
Copy link
Contributor

Which issue(s) does this pull-request address?

Part of: #21

Description

One small step towards "Reduce number of files in project root #21" issue. Putting prettier settings in the package.json eliminates one file and works with IDEs, CLI, our yarn format[:fix], etc. Some folks might prefer a separate file, but this seems pretty reasonable.

Checklist

  • I have read the contributor guide
  • Linting passes: yarn lint or yarn lint:fix
  • Formatting is applied: yarn format or yarn format:fix
  • Unit tests pass: yarn test:unit
  • E2E tests pass: yarn test:e2e
  • Tests are included
  • Documentation is changed or added
  • Commit messages and PR title follow conventional commits

One small step towards "Reduce number of files in project root i-am-bee#21" issue.
Putting prettier settings in the package.json eliminates one file and works
with IDEs, CLI, our yarn format[:fix], etc...

Ref: i-am-bee#21

Signed-off-by: Mark Sturdevant <mark.sturdevant@ibm.com>
Copy link
Contributor

@Tomas2D Tomas2D left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏻

@Tomas2D Tomas2D merged commit c5b61be into i-am-bee:main Sep 19, 2024
1 check passed
abughali pushed a commit to abughali/bee-agent-framework that referenced this pull request Sep 22, 2024
One small step towards "Reduce number of files in project root i-am-bee#21" issue.
Putting prettier settings in the package.json eliminates one file and works
with IDEs, CLI, our yarn format[:fix], etc...

Ref: i-am-bee#21
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.

2 participants