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

Disable scaladoc during packaging for scala 2.10 #959

Merged

Conversation

mikejcurry
Copy link
Contributor

My previous PR #958 missed the fact that the shell script which executes the build runs a publish as the final command. In the case of Pull Requests this is a publishLocal. On master it runs something different, which appears to publish snapshots.

As such, the previous PR allowed #898 to pass the site generation process, but it failed during the publish stage - the last command executed during the pull request build process.

Sorry about that.

This one disables the scaladoc packaging phase during publish for 2.10, which hopefully finally allows #898 to pass.

I tested publishLocal and it passes fine on 2.10 now albeit with docs not generated. 2.11 still generates scaladoc during the publish phase.

Disables scaladoc generation during publish phase for scala 2.10
@codecov-io
Copy link

Current coverage is 89.78%

Merging #959 into master will not affect coverage as of 34abc24

@@            master    #959   diff @@
======================================
  Files          180     180       
  Stmts         2144    2144       
  Branches        42      42       
  Methods          0       0       
======================================
  Hit           1925    1925       
  Partial          0       0       
  Missed         219     219       

Review entire Coverage Diff as of 34abc24

Powered by Codecov. Updated on successful CI builds.

@ceedubs
Copy link
Contributor

ceedubs commented Mar 28, 2016

👍 thanks!

@mikejcurry
Copy link
Contributor Author

Anybody else happy to give this a thumbs up?

It is not a perfect solution, and there might be a better way to disable scaladoc for 2.10 on these projects but it allows scaladoc to be written for more type classes, with Comonad being an example in point.

@milessabin
Copy link
Member

👍

@milessabin milessabin merged commit 6154e91 into typelevel:master Apr 2, 2016
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.

5 participants