Skip to content

Fix publisher handle variable name #202

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jun 13, 2022
Merged

Fix publisher handle variable name #202

merged 1 commit into from
Jun 13, 2022

Conversation

esteve
Copy link
Collaborator

@esteve esteve commented Jun 13, 2022

This fixes main not building due to #194

@esteve esteve requested a review from a team June 13, 2022 14:06
@jhdcs
Copy link
Collaborator

jhdcs commented Jun 13, 2022

The tests appear to be failing from an internal server error where the repositories are stored. Should we wait for that to get
resolved before merging?

@esteve
Copy link
Collaborator Author

esteve commented Jun 13, 2022

@jhdcs @nnmm thanks for the prompt reviews, I'll wait until CI passes, but it looks like there's an issue with docker.io being down.

@esteve esteve merged commit df78ac7 into main Jun 13, 2022
@delete-merged-branch delete-merged-branch bot deleted the fix-handle branch June 13, 2022 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nnmm nnmm nnmm approved these changes

@jhdcs jhdcs jhdcs approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@esteve @jhdcs @nnmm