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

Create docker-publish.yml #188

Merged
merged 1 commit into from
Mar 31, 2024
Merged

Create docker-publish.yml #188

merged 1 commit into from
Mar 31, 2024

Conversation

drzo
Copy link
Contributor

@drzo drzo commented Dec 29, 2023

No description provided.

@linas linas merged commit 7ab31c2 into opencog:master Mar 31, 2024
@linas
Copy link
Member

linas commented Mar 31, 2024

Thank you!
My apologies for merging this so late; I simply did not see the pull request sitting in the queue. Maybe it got lost in email, I dunno.

@linas
Copy link
Member

linas commented Apr 10, 2024

Actually, I'm now going to revert this. It spews an endless list of errors. There are 6 or 7 images that need to be built; this script seems to assume that there is only one. The existing file publish-images.yml is configured to do the right thing.

linas added a commit that referenced this pull request Apr 10, 2024
The script in #188 seems to be broken. It assumes only one docker file,
when there are 6 or 7 of them. I can't figure out ow to fix it.
The publish-images.yml already does the right thing, more or less.
@linas linas mentioned this pull request Apr 10, 2024
linas added a commit that referenced this pull request Apr 10, 2024
@linas
Copy link
Member

linas commented Apr 10, 2024

Reverted in #192

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