search

src-d / awesome-machine-learning-on-source-code

Cool links & research papers related to Machine Learning applied to source code (MLonCode)
Creative Commons Attribution Share Alike 4.0 International
6.18k stars 842 forks source link

Removed broken link #128

Closed thecodingaviator closed 5 years ago

thecodingaviator commented 5 years ago

Removed broken link from README

thecodingaviator commented 5 years ago

Yup, will do today

thecodingaviator commented 5 years ago

@EgorBu Done!

m09 commented 5 years ago

Hi @thecodingaviator, thanks for the PR! We'd need a DCO before merging. You can get more infos about it here

thecodingaviator commented 5 years ago

@m09 I'm not sure how to add DCO, do I have to pull my fork from GitHub, make a change, commit with the message - Signed-off-by: Random J Developer <random@developer.example.org> , push it to my fork and open a new PR?

smola commented 5 years ago

@thecodingaviator No need to create a new PR. You can update this one with sign-off as follows:

git clone git@github.com:thecodingaviator/awesome-machine-learning-on-source-code.git
cd awesome-machine-learning-on-source-code
git checkout patch-1
git commit --amend --no-edit --signoff
git push origin +patch-1

This will clone your repo, checkout the branch from the PR (patch-1), change the last commit to add the sign-off message, then push to the same branch overwriting it. When changing the branch patch-1, this PR will update automatically.

thecodingaviator commented 5 years ago

@smola Done