Skip to content
This repository has been archived by the owner on Mar 10, 2023. It is now read-only.

Fixing badge link #701

Merged
merged 1 commit into from
Dec 1, 2020
Merged

Fixing badge link #701

merged 1 commit into from
Dec 1, 2020

Conversation

spekulatius
Copy link
Contributor

Hey @alexellis

just fixing the badge link...

Cheers,
Peter

@derek
Copy link

derek bot commented Dec 1, 2020

Thank you for your contribution. I've just checked and your commit doesn't appear to be signed-off. That's something we need before your Pull Request can be merged. Please see our contributing guide.
Tip: if you only have one commit so far then run: git commit --amend --signoff and then git push --force.

@spekulatius
Copy link
Contributor Author

It really works as demonstrated in the video 👍

Signed-off-by: Peter Thaleikis <peter.thaleikis@gmail.com>
@derek derek bot removed the no-dco label Dec 1, 2020
Copy link
Member

@alexellis alexellis left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you for tidying this up.

@alexellis
Copy link
Member

Thanks for the PR. Which video did you see @spekulatius?

Do you want to join us on Slack to talk about OFC?

@alexellis
Copy link
Member

/msg: slack

@derek
Copy link

derek bot commented Dec 1, 2020

--
Join Slack to connect with the community
https://docs.openfaas.com/community

@alexellis alexellis merged commit 2f2413a into openfaas:master Dec 1, 2020
@spekulatius
Copy link
Contributor Author

I've actually seen a few by now. I think it was this one: https://www.youtube.com/watch?v=ibPwVggXAFI

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants