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 public pool names #547

Merged
merged 1 commit into from
Sep 7, 2022
Merged

Update public pool names #547

merged 1 commit into from
Sep 7, 2022

Conversation

wtgodbe
Copy link
Contributor

@wtgodbe wtgodbe commented Sep 1, 2022

No description provided.

@wtgodbe wtgodbe requested a review from dougbu September 1, 2022 22:08
@wtgodbe
Copy link
Contributor Author

wtgodbe commented Sep 1, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines failed to run 1 pipeline(s).

@wtgodbe
Copy link
Contributor Author

wtgodbe commented Sep 2, 2022

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@alexperovich
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@alexperovich
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@wtgodbe
Copy link
Contributor Author

wtgodbe commented Sep 7, 2022

@alexperovich do we need the same changes here as in aspnet/AspNetWebHooks#154?

@alexperovich
Copy link
Collaborator

Yea, we do.

@wtgodbe
Copy link
Contributor Author

wtgodbe commented Sep 7, 2022

Actually, wait, I don't see anywhere where we set an endpoint in a .yml file - is it something else?

@wtgodbe wtgodbe closed this Sep 7, 2022
@wtgodbe wtgodbe reopened this Sep 7, 2022
@alexperovich
Copy link
Collaborator

Not that change. I need to update the build to use the workaround so I need perms to change the webhooks.

@wtgodbe
Copy link
Contributor Author

wtgodbe commented Sep 7, 2022

I gave you admin rights

@alexperovich alexperovich reopened this Sep 7, 2022
@alexperovich
Copy link
Collaborator

Okay, it looks fixed now.

@wtgodbe wtgodbe merged commit ad32500 into main Sep 7, 2022
@wtgodbe wtgodbe deleted the wtgodbe/NetCorePub branch September 7, 2022 22:30
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