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

fix: prettier errors #106

Merged
merged 5 commits into from
Apr 16, 2024
Merged

Conversation

wilsonianb
Copy link
Contributor

Pull Request

Description

Fix prettier errors breaking husky pre-commit hook

Add general CI GitHub Actions workflow

Checklist

Please review and check the following items to ensure your pull request can be processed smoothly:

  • My code adheres to the project's coding standards and style guidelines.

  • I have tested my changes thoroughly, including unit tests where applicable.

  • I have updated the documentation to reflect any changes made in this pull request.

  • My branch is up-to-date with the latest changes from the main branch.

  • I have received code review feedback and addressed any comments or concerns.

Additional Notes

If there are any additional details or considerations that reviewers should be aware of, please provide them here.

References, Fixes and Closes

List any other pull requests or issues that this pull request references, fixes, or closes, if applicable.

  • References #related_issue_number
  • Fixes #related_issue_number
  • Closes #related_issue_number

Copy link

vercel bot commented Feb 27, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
agentprotocolai ⬜️ Ignored (Inspect) Visit Preview Mar 25, 2024 9:13pm

@wilsonianb
Copy link
Contributor Author

I'm not sure what's up with the build error.

/home/runner/work/agent-protocol/agent-protocol/packages/sdk/js/src/agent.ts
Error:   318:19  error  Unexpected any value in conditional. An explicit comparison or type cast is required  @typescript-eslint/strict-boolean-expressions

npm run ci works for me locally...

@wilsonianb wilsonianb marked this pull request as ready for review March 25, 2024 21:26
@jzanecook jzanecook merged commit ca361e1 into AI-Engineer-Foundation:main Apr 16, 2024
4 checks passed
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