-
Notifications
You must be signed in to change notification settings - Fork 99
Open
Labels
good first issueGood for newcomersGood for newcomerspriority/minorIndicates the PR or issue related to non-urgent bugs, features, and improvements.Indicates the PR or issue related to non-urgent bugs, features, and improvements.tech debt[Issue Type] User-unaware issues, such as code refactor and infrastructure maintenance.[Issue Type] User-unaware issues, such as code refactor and infrastructure maintenance.
Description
Search before asking
- I searched in the issues and found nothing similar.
Description
We provide easy-to-use constant strings for resource class names in ResourceName.java and resource.py. Two issues were observed in practice.
- The constant strings might be inconsistent with the actual class name. E.g., [hotfix] Fix Azure ChatModel class path in ResourceName. #562
- Developers tend to forget adding resource names for cross-language resource references, i.e.
ResourceName.Pythonon the java side andResourceName.Javaon the python side.
It would be nice check in CI that the referenced class actually exist, and names in Java and Python are consistent.
Are you willing to submit a PR?
- I'm willing to submit a PR!
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomerspriority/minorIndicates the PR or issue related to non-urgent bugs, features, and improvements.Indicates the PR or issue related to non-urgent bugs, features, and improvements.tech debt[Issue Type] User-unaware issues, such as code refactor and infrastructure maintenance.[Issue Type] User-unaware issues, such as code refactor and infrastructure maintenance.