-
-
Notifications
You must be signed in to change notification settings - Fork 714
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: update tools.json #3474
chore: update tools.json #3474
Conversation
WalkthroughThe pull request introduces modifications to two configuration files: Changes
Suggested labels
Poem
Tip CodeRabbit's docstrings feature is now available as part of our Early Access Program! Simply use the command Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
✅ Deploy Preview for asyncapi-website ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3474 +/- ##
=======================================
Coverage 86.59% 86.59%
=======================================
Files 21 21
Lines 664 664
=======================================
Hits 575 575
Misses 89 89 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 3
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
config/all-tags.json
(1 hunks)config/tools-automated.json
(12 hunks)
🔇 Additional comments (2)
config/all-tags.json (1)
1-1
: LGTM! Color standardization improves visual consistency
The changes standardize the color scheme for multiple technology entries while maintaining proper JSON structure. The addition of AsyncAPI CLI follows the established pattern.
config/tools-automated.json (1)
74-95
: LGTM! Appropriate multi-category placement for Zod Sockets
The tool is correctly added to relevant categories with consistent information across entries.
Also applies to: 292-313
"language": [ | ||
"javascript" | ||
], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix incorrect language specifications in templates
Several Java-based templates incorrectly specify "javascript" as their language. Additionally, there's inconsistent usage of string vs array for the language field.
Apply these corrections:
"filters": {
"language": [
- "javascript"
+ "java"
],
Also applies to: 784-786, 806-808
{ | ||
"title": "AsyncAPI CLI", | ||
"description": "One CLI to rule them all. \nThis is a CLI that aims to integrate all AsyncAPI tools that you need while AsyncAPI document development and maintainance. \nYou can use it to generate docs or code, validate AsyncAPI document and event create new documents.\n", | ||
"links": { | ||
"websiteUrl": "https://www.asyncapi.com/tools/cli", | ||
"repoUrl": "https://github.com/asyncapi/cli" | ||
}, | ||
"filters": { | ||
"technology": [ | ||
"TypeScript" | ||
], | ||
"categories": [ | ||
"others", | ||
"cli" | ||
], | ||
"hasCommercial": false, | ||
"isAsyncAPIOwner": true | ||
} | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consolidate duplicate AsyncAPI CLI entries
There are duplicate entries for AsyncAPI CLI pointing to different repositories (official and fork). This could cause confusion for users.
Consider:
- Removing the fork entry if it's not officially supported
- Clearly indicating in the description if the fork serves a different purpose
Also applies to: 847-874
"title": "GitHub Action for CLI", | ||
"description": "GitHub Action with generator, validator, converter and others - all in one for your AsyncAPI documents with AsyncAPI CLI as backbone", | ||
"links": { | ||
"repoUrl": "https://github.com/actions-marketplace-validations/asyncapi_github-action-for-generator" | ||
"repoUrl": "https://github.com/asyncapi/github-action-for-cli" | ||
}, | ||
"filters": { | ||
"technology": [ | ||
"AsyncAPI Generator" | ||
"AsyncAPI CLI" | ||
], | ||
"categories": [ | ||
"github-action" | ||
], | ||
"hasCommercial": false, | ||
"isAsyncAPIOwner": false | ||
"isAsyncAPIOwner": true | ||
} | ||
}, | ||
{ | ||
"title": "GitHub Action for CLI", | ||
"description": "GitHub Action with generator, validator, converter and others - all in one for your AsyncAPI documents with AsyncAPI CLI as backbone", | ||
"title": "GitHub Action for Generator", | ||
"description": "CLI to work with your AsyncAPI files. You can validate them and in the future use a generator and even bootstrap a new file. Contributions are welcomed!", | ||
"links": { | ||
"repoUrl": "https://github.com/asyncapi/github-action-for-cli" | ||
"repoUrl": "https://github.com/asyncapi/cli" | ||
}, | ||
"filters": { | ||
"technology": [ | ||
"AsyncAPI CLI" | ||
"AsyncAPI Generator" | ||
], | ||
"categories": [ | ||
"github-action" | ||
"github-actions" | ||
], | ||
"hasCommercial": false, | ||
"isAsyncAPIOwner": true | ||
} | ||
}, | ||
{ | ||
"title": "GitHub Action for Generator", | ||
"description": "CLI to work with your AsyncAPI files. You can validate them and in the future use a generator and even bootstrap a new file. Contributions are welcomed!", | ||
"description": null, | ||
"links": { | ||
"repoUrl": "https://github.com/asyncapi/cli" | ||
"repoUrl": "https://github.com/actions-marketplace-validations/asyncapi_github-action-for-generator" | ||
}, | ||
"filters": { | ||
"technology": [ | ||
"AsyncAPI Generator" | ||
], | ||
"categories": [ | ||
"github-actions" | ||
"github-action" | ||
], | ||
"hasCommercial": false, | ||
"isAsyncAPIOwner": true | ||
"isAsyncAPIOwner": false | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix inconsistent category naming in GitHub Actions
The category name varies between "github-action" and "github-actions". This should be standardized to ensure consistent filtering.
Standardize to "github-action" across all entries:
"categories": [
- "github-actions"
+ "github-action"
],
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
"title": "GitHub Action for CLI", | |
"description": "GitHub Action with generator, validator, converter and others - all in one for your AsyncAPI documents with AsyncAPI CLI as backbone", | |
"links": { | |
"repoUrl": "https://github.com/actions-marketplace-validations/asyncapi_github-action-for-generator" | |
"repoUrl": "https://github.com/asyncapi/github-action-for-cli" | |
}, | |
"filters": { | |
"technology": [ | |
"AsyncAPI Generator" | |
"AsyncAPI CLI" | |
], | |
"categories": [ | |
"github-action" | |
], | |
"hasCommercial": false, | |
"isAsyncAPIOwner": false | |
"isAsyncAPIOwner": true | |
} | |
}, | |
{ | |
"title": "GitHub Action for CLI", | |
"description": "GitHub Action with generator, validator, converter and others - all in one for your AsyncAPI documents with AsyncAPI CLI as backbone", | |
"title": "GitHub Action for Generator", | |
"description": "CLI to work with your AsyncAPI files. You can validate them and in the future use a generator and even bootstrap a new file. Contributions are welcomed!", | |
"links": { | |
"repoUrl": "https://github.com/asyncapi/github-action-for-cli" | |
"repoUrl": "https://github.com/asyncapi/cli" | |
}, | |
"filters": { | |
"technology": [ | |
"AsyncAPI CLI" | |
"AsyncAPI Generator" | |
], | |
"categories": [ | |
"github-action" | |
"github-actions" | |
], | |
"hasCommercial": false, | |
"isAsyncAPIOwner": true | |
} | |
}, | |
{ | |
"title": "GitHub Action for Generator", | |
"description": "CLI to work with your AsyncAPI files. You can validate them and in the future use a generator and even bootstrap a new file. Contributions are welcomed!", | |
"description": null, | |
"links": { | |
"repoUrl": "https://github.com/asyncapi/cli" | |
"repoUrl": "https://github.com/actions-marketplace-validations/asyncapi_github-action-for-generator" | |
}, | |
"filters": { | |
"technology": [ | |
"AsyncAPI Generator" | |
], | |
"categories": [ | |
"github-actions" | |
"github-action" | |
], | |
"hasCommercial": false, | |
"isAsyncAPIOwner": true | |
"isAsyncAPIOwner": false | |
} | |
} | |
"title": "GitHub Action for CLI", | |
"description": "GitHub Action with generator, validator, converter and others - all in one for your AsyncAPI documents with AsyncAPI CLI as backbone", | |
"links": { | |
"repoUrl": "https://github.com/asyncapi/github-action-for-cli" | |
}, | |
"filters": { | |
"technology": [ | |
"AsyncAPI CLI" | |
], | |
"categories": [ | |
"github-action" | |
], | |
"hasCommercial": false, | |
"isAsyncAPIOwner": true | |
} | |
}, | |
{ | |
"title": "GitHub Action for Generator", | |
"description": "CLI to work with your AsyncAPI files. You can validate them and in the future use a generator and even bootstrap a new file. Contributions are welcomed!", | |
"links": { | |
"repoUrl": "https://github.com/asyncapi/cli" | |
}, | |
"filters": { | |
"technology": [ | |
"AsyncAPI Generator" | |
], | |
"categories": [ | |
"github-action" | |
], | |
"hasCommercial": false, | |
"isAsyncAPIOwner": true | |
} | |
}, | |
{ | |
"title": "GitHub Action for Generator", | |
"description": null, | |
"links": { | |
"repoUrl": "https://github.com/actions-marketplace-validations/asyncapi_github-action-for-generator" | |
}, | |
"filters": { | |
"technology": [ | |
"AsyncAPI Generator" | |
], | |
"categories": [ | |
"github-action" | |
], | |
"hasCommercial": false, | |
"isAsyncAPIOwner": false | |
} | |
} |
⚡️ Lighthouse report for the changes in this PR:
Lighthouse ran on https://deploy-preview-3474--asyncapi-website.netlify.app/ |
Automated changes by create-pull-request GitHub action
Summary by CodeRabbit
New Features
Updates