Skip to content

chore: add format check to typescript packages - #254

Merged
0xRAG merged 1 commit into
masterfrom
AGENT-111
Feb 4, 2025
Merged

0xRAG merged 1 commit into
masterfrom
AGENT-111

Conversation

@0xRAG

@0xRAG 0xRAG commented Feb 4, 2025

Copy link
Copy Markdown
Contributor

What changed? Why?

Adds check that will fail CI if TS code is not formatted correctly

Qualified Impact

@cb-heimdall

cb-heimdall commented Feb 4, 2025 •

Copy link
Copy Markdown

✅ Heimdall Review Status

Requirement Status More Info
Reviews ✅ 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

run: poetry run make lint

lint-agentkit-typescript:
lint-typescript:

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This will run against all of our TS packages, including the examples

@@ -0,0 +1,7 @@
docs/

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

These files don't need to be copy-pasted into each package, but I'm holding off on creating a shareable config until we lock down the monorepo management tool we end up going with

@0xRAG
0xRAG merged commit b006ddd into master Feb 4, 2025
@0xRAG
0xRAG deleted the AGENT-111 branch February 4, 2025 17:49
emiridbest pushed a commit to emiridbest/agentkit that referenced this pull request Feb 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants