Update README.md - Githubissues

vyaspranjal33 commented 6 years ago

beautified link

m09 commented 6 years ago

Hi Pranjal, thanks for the PR. Two things:

we'll have to close this one, the LICENSE link would be broken if we merged, and it is markdown so we can't actually rename to remove the .md and consider it an improvement
when you propose a PR, it is submitted to our travis CI pool of 5 workers. Since you proposed a lot of small PRs this pool of workers is getting drown in the runs needed to evaluate your changes. Because of this we'd like to ask you to actually stop submitting PRs since it prevents more involved changes to be evaluated. We know it is the last day of Hacktoberfest but we hope you'll understand. Thanks!

vyaspranjal33 commented 6 years ago

@m09 i can fix if u allow and consider

m09 commented 6 years ago

As I said the link is fine: the license is actually markdown and having the .md in the link shows that. Also please consider the second point I mentioned which is really slowing down our teams currently. Thanks!

vyaspranjal33 commented 6 years ago

i wont be sending more PRs from now @m09 please consider these PRs , i will revert to .md extension :) hope u understand :sweat:

src-d / datasets

Update README.md #90