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

fix: dde-open error when use old am #111

Merged
merged 1 commit into from
Nov 26, 2023
Merged

Conversation

kt286
Copy link
Contributor

@kt286 kt286 commented Nov 25, 2023

fix: dde-open error when use old am

@deepin-ci-robot
Copy link

Hi @kt286. Thanks for your PR.:smiley:

@kt286
Copy link
Contributor Author

kt286 commented Nov 26, 2023

/check obs

@deepin-bot
Copy link
Contributor

deepin-bot bot commented Nov 26, 2023

Deepin Obs Bot
Deepin redeliveried failed, need update or recreate pull request!!!

@Decodetalkers
Copy link
Contributor

更新下Gomod

dde-open/main.go Outdated Show resolved Hide resolved
@Decodetalkers
Copy link
Contributor

只要修老Am的部分,Commit和标题也改下,是修老Am的情况

@kt286 kt286 force-pushed the fix/new-am branch 2 times, most recently from 956e86d to 5c8e3fb Compare November 26, 2023 04:13
@kt286 kt286 changed the title feat: dde-open use new am fix: dde-open error when use old am Nov 26, 2023
dde-open error when use old am
@Decodetalkers
Copy link
Contributor

Commit Lint挂掉了,最下面加一个Log:就可以了

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Decodetalkers, kt286

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@Decodetalkers
Copy link
Contributor

Issue: linuxdeepin/developer-center#6280

@kt286
Copy link
Contributor Author

kt286 commented Nov 26, 2023

/merge

@deepin-bot deepin-bot bot merged commit c39c19c into linuxdeepin:master Nov 26, 2023
12 checks passed
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