-
Notifications
You must be signed in to change notification settings - Fork 4
feat: Expose queue positions for runs in both the launchpad and the CLI #318
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -176,7 +176,8 @@ codegen: | |
| find codegen/out/aignx/codegen/models/ -name "[a-z]*.py" -type f | sed 's|.*/\(.*\)\.py|\1|' | xargs -I{} echo "from .{} import *" > codegen/out/aignx/codegen/models/__init__.py | ||
| # fix resource patch | ||
| # in codegen/out/public_api.py replace all occurrences of resource_path='/v1 with resource_path='/api/v1 | ||
| sed -i '' "s|resource_path='/v1|resource_path='/api/v1|g" codegen/out/aignx/codegen/api/public_api.py | ||
| # Use portable sed syntax: -i'' works on both macOS and Linux | ||
| sed -i"" "s|resource_path='/v1|resource_path='/api/v1|g" codegen/out/aignx/codegen/api/public_api.py | ||
|
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Command was failing on Linux. Claude says this should now work on both Mac and Linux. |
||
|
|
||
| # Special rule to catch any arguments (like patch, minor, major, pdf, Python versions, or x.y.z) | ||
| # This prevents "No rule to make target" errors when passing arguments to make commands | ||
|
|
||
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,34 +1,35 @@ | ||
| from .item_result_read_response import * | ||
| from .run_output import * | ||
| from .artifact_state import * | ||
| from .validation_error_loc_inner import * | ||
| from .run_state import * | ||
| from .item_output import * | ||
| from .run_creation_response import * | ||
| from .output_artifact_visibility import * | ||
| from .organization_read_response import * | ||
| from .me_read_response import * | ||
| from .run_item_statistics import * | ||
| from .validation_error import * | ||
| from .application_read_response import * | ||
| from .application_read_short_response import * | ||
| from .output_artifact_result_read_response import * | ||
| from .version_read_response import * | ||
| from .item_output import * | ||
| from .run_termination_reason import * | ||
| from .run_creation_response import * | ||
| from .output_artifact_scope import * | ||
| from .me_read_response import * | ||
| from .run_output import * | ||
| from .item_result_read_response import * | ||
| from .application_version import * | ||
| from .run_read_response import * | ||
| from .run_state import * | ||
| from .input_artifact_creation_request import * | ||
| from .item_creation_request import * | ||
| from .custom_metadata_update_response import * | ||
| from .input_artifact import * | ||
| from .custom_metadata_update_request import * | ||
| from .item_state import * | ||
| from .auth0_organization import * | ||
| from .application_version import * | ||
| from .http_validation_error import * | ||
| from .artifact_state import * | ||
| from .application_read_short_response import * | ||
| from .output_artifact import * | ||
| from .user_read_response import * | ||
| from .run_termination_reason import * | ||
| from .input_artifact import * | ||
| from .output_artifact_result_read_response import * | ||
| from .version_read_response import * | ||
| from .item_termination_reason import * | ||
| from .artifact_output import * | ||
| from .auth0_organization import * | ||
| from .auth0_user import * | ||
| from .run_read_response import * | ||
| from .item_creation_request import * | ||
| from .http_validation_error import * | ||
| from .validation_error_loc_inner import * | ||
| from .artifact_termination_reason import * | ||
| from .run_creation_request import * | ||
| from .item_termination_reason import * | ||
| from .run_item_statistics import * | ||
| from .output_artifact_visibility import * | ||
| from .custom_metadata_update_request import * | ||
| from .artifact_output import * | ||
| from .output_artifact import * | ||
| from .application_read_response import * |
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.
Unrelated improvement: The portable sed syntax fix (
-i""vs-i '') is a good improvement but should be in a separate commit:git commit -m "fix(codegen): use portable sed syntax for macOS and Linux compatibility"Context: The
-i ''syntax only works on macOS, while-i""works on both macOS and Linux. This is a legitimate cross-platform fix but belongs in its own commit per single-responsibility principle.