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 the reference of tailwindConfig #95

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

guyue66
Copy link

@guyue66 guyue66 commented Oct 18, 2022

feat: add the reference of tailwindConfig and resolve the config conflick

@netlify
Copy link

netlify bot commented Oct 18, 2022

Deploy Preview for steady-malasada-9c6b15 ready!

Name Link
🔨 Latest commit e92b86b
🔍 Latest deploy log https://app.netlify.com/sites/steady-malasada-9c6b15/deploys/634e6d84bedf070009286993
😎 Deploy Preview https://deploy-preview-95--steady-malasada-9c6b15.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@guyue66 guyue66 marked this pull request as draft October 18, 2022 07:46
@guyue66 guyue66 closed this Oct 18, 2022
@guyue66
Copy link
Author

guyue66 commented Oct 18, 2022

forget commit -s

Signed-off-by: yifu hu <[email protected]>
@guyue66 guyue66 reopened this Oct 18, 2022
@IronCore864
Copy link
Member

forget commit -s

Hi, you can fix this by "git commit -s --amend", then push force to your own branch.

Since we are a CNCF project now, we should comply with the rules, and signing the DCO is one of them.

Thanks for the contribution!

@IronCore864
Copy link
Member

forget commit -s

Also, could you please create an issue and link this PR to the issue? So that the reviewers know exactly what problem this PR tries to solve. Thanks!

@KeHaohaoke
Copy link
Collaborator

@guyue66
Hi, please take care of the git commit signature. Otherwise it won't pass the relevant CI checks.

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