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

[bugfix](ci) fix phoenix ci error #5530

Merged
merged 1 commit into from
Sep 20, 2023
Merged

Conversation

liugddx
Copy link
Member

@liugddx liugddx commented Sep 20, 2023

Purpose of this pull request

change driver download url

Does this PR introduce any user-facing change?

No

How was this patch tested?

passed the ci

Check list

@Hisoka-X
Copy link
Member

nit: As a committer, I think we need to be more detailed in the PR description, if we are not doing this, why ask other contributors to do the same thing
image

@liugddx
Copy link
Member Author

liugddx commented Sep 20, 2023

nit: As a committer, I think we need to be more detailed in the PR description, if we are not doing this, why ask other contributors to do the same thing image

My fault, I will pay attention.

@Hisoka-X
Copy link
Member

My fault, I will pay attention.

Never mind! Just friendly tip! :)

@EricJoy2048 EricJoy2048 merged commit bd2024b into apache:dev Sep 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants