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

feat: add scripts, frontend and assets - 阳历新年快乐! #49

Merged
merged 1 commit into from
Dec 31, 2023

Conversation

Young-Lord
Copy link
Contributor

No description provided.

CanoKey Pigeon 的第一个签名献给 Dress(
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

你好 @Young-Lord,感谢提交PR!
Hi @Young-Lord, thanks for opening the Pull Request!
我们会尽早审核回应😊
We will respond to it as soon as possible😊

Copy link
Member

@CuteDress CuteDress left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

好!

@CuteDress CuteDress merged commit efcf7ca into Cute-Dress:master Dec 31, 2023
1 check passed
@Young-Lord
Copy link
Contributor Author

为什么 gpg 签名掉了……貌似该用Create a merge commit的……
虽然这样也行就是了

@CuteDress
Copy link
Member

为什么 gpg 签名掉了……貌似该用Create a merge commit的…… 虽然这样也行就是了

因为用的是 squash merge

@Young-Lord
Copy link
Contributor Author

Young-Lord commented Jan 1, 2024

个人认为要是能 fast-forward 就最好了,奈何 GitHub 不支持
但是用一个 GitHub Actions 是可以做到的

@Young-Lord
Copy link
Contributor Author

@Young-Lord
Copy link
Contributor Author

完整实现在这里,介意我开个 pr 吗?
https://github.com/Young-Lord/ci-runner-for-sth/blob/main/.github/workflows/test-perm.yml

@CuteDress
Copy link
Member

完整实现在这里,介意我开个 pr 吗? https://github.com/Young-Lord/ci-runner-for-sth/blob/main/.github/workflows/test-perm.yml

可以啊

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.

2 participants