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

Contributing guidelines link #2576

Merged
merged 1 commit into from
Oct 31, 2018

Conversation

fmsbeekmans
Copy link
Contributor

Made the link to the code guidelines in CONTRIBUTING.md absolute so that it works in both the github markdown viewer and form the website as discussed here https://gitter.im/typelevel/cats?at=5bccdbb8bbdc0b25057a0716.

Consistent with the references to laws lawtesting.

@ceedubs
Copy link
Contributor

ceedubs commented Oct 21, 2018

@fmsbeekmans thanks!

This is a welcome change, but I think that you accidentally included some unrelated Chain changes. Also it looks like for those Chain changes you'll want to run fmt to apply scalafmt formatting.

@kailuowang
Copy link
Contributor

Thanks! Would you separate out the distinct addition to a new PR?

@fmsbeekmans
Copy link
Contributor Author

@ceedubs rebased them out.

@fmsbeekmans
Copy link
Contributor Author

@ceedubs @kailuowang Thanks for the feedback, I've done the formatting and created a PR #2579

@codecov-io
Copy link

codecov-io commented Oct 22, 2018

Codecov Report

Merging #2576 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2576   +/-   ##
=======================================
  Coverage   95.14%   95.14%           
=======================================
  Files         361      361           
  Lines        6630     6630           
  Branches      289      283    -6     
=======================================
  Hits         6308     6308           
  Misses        322      322

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9069dbb...603a561. Read the comment docs.

Copy link
Contributor

@ceedubs ceedubs left a comment

Choose a reason for hiding this comment

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

Great, thanks @fmsbeekmans!

@LukaJCB LukaJCB merged commit a26869d into typelevel:master Oct 31, 2018
@kailuowang kailuowang added this to the 1.5 milestone Oct 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants