Skip to content

Test: do not merge - ignore - #3036

Draft
poornas2505 wants to merge 3 commits into
keras-team:masterfrom
poornas2505:master
Draft

Test: do not merge - ignore#3036
poornas2505 wants to merge 3 commits into
keras-team:masterfrom
poornas2505:master

Conversation

@poornas2505

Copy link
Copy Markdown

Approved issue link

Description of the change

Reference

Colab Notebook

Checklist

  • I have added all the necessary unit tests for my change.
  • I have verified that my change does not break existing code and works with all backends (TensorFlow, JAX, and PyTorch).
  • My PR is based on the latest changes of the main branch (if unsure, rebase the code).
  • I have followed the Keras Hub Model contribution guidelines in making these changes.
  • I have followed the Keras Hub API design guidelines in making these changes.
  • I have signed the Contributor License Agreement.

@github-actions
github-actions Bot marked this pull request as draft September 4, 2026 22:47
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

❌ Approved issue check failed. This PR stays in draft until it links an approved issue that is assigned to you.

To fix this:

  1. Find or open an issue for this change and ask a maintainer to approve it and assign it to you.
  2. Link it under the "Approved issue link" section of this PR's description, e.g. Fixes #123.

The PR will be marked Ready for review automatically once the check passes.

No issue reference was found in the description.

@google-cla

google-cla Bot commented Sep 4, 2026

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a syntax error in the test file keras_hub/src/tests/import_test.py by corrupting the import statement to hjsujsnimport ast. The reviewer correctly identified this issue and provided a code suggestion to revert the change to import ast.

@@ -1,4 +1,4 @@
import ast
hjsujsnimport ast

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

critical

This line contains a syntax error ('hjsujsnimport ast') which will prevent the test file from being parsed or executed. It should be reverted to 'import ast'.

Suggested change
hjsujsnimport ast
import ast

@laxmareddyp laxmareddyp closed this Sep 4, 2026
@laxmareddyp laxmareddyp reopened this Sep 4, 2026
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