Skip to content

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

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
EricJoy2048 merged 1 commit into from
Sep 20, 2023
Merged

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

EricJoy2048 merged 1 commit into from
Sep 20, 2023

Conversation

@liugddx
Copy link
Member

@liugddx liugddx commented Sep 20, 2023 •
edited
Loading

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

@liugddx liugddx added the CI&CD label Sep 20, 2023
@Hisoka-X
Copy link
Member

Hisoka-X 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

@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

Hisoka-X commented Sep 20, 2023

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

Reviewers

@EricJoy2048 EricJoy2048 EricJoy2048 approved these changes

@Hisoka-X Hisoka-X Hisoka-X approved these changes

Assignees

No one assigned

Labels

approved CI&CD reviewed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@liugddx @Hisoka-X @EricJoy2048