Skip to content
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

Update CI to use keras-core from pypi #1124

Merged
merged 1 commit into from
Jul 10, 2023
Merged

Update CI to use keras-core from pypi #1124

merged 1 commit into from
Jul 10, 2023

Conversation

mattdangerw
Copy link
Member

No description provided.

Copy link
Contributor

@jbischof jbischof left a comment

Choose a reason for hiding this comment

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

🍾

@mattdangerw mattdangerw requested a review from ianstenbit July 10, 2023 20:06
@mattdangerw mattdangerw merged commit 0a91540 into core Jul 10, 2023
@mattdangerw mattdangerw deleted the core-ci branch July 10, 2023 20:46
mattdangerw added a commit to mattdangerw/keras-hub that referenced this pull request Jul 10, 2023
In keras-team#1124, I accidentally added a line doing a `pip install .` package
install to the publish flow.

This is silly, we shouldn't need to be installing the package before
building the package for pypi, and it appears to be causing failures
on the action:
https://github.com/keras-team/keras-nlp/actions/runs/5513665846/jobs/10052068366

These were not caught because we don't run the pypi action on each PR.
mattdangerw added a commit that referenced this pull request Jul 10, 2023
In #1124, I accidentally added a line doing a `pip install .` package
install to the publish flow.

This is silly, we shouldn't need to be installing the package before
building the package for pypi, and it appears to be causing failures
on the action:
https://github.com/keras-team/keras-nlp/actions/runs/5513665846/jobs/10052068366

These were not caught because we don't run the pypi action on each PR.
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.

3 participants