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

Laika site support for sbt 1.0 #125

Merged
merged 13 commits into from
Feb 12, 2018
Merged

Laika site support for sbt 1.0 #125

merged 13 commits into from
Feb 12, 2018

Conversation

jonas
Copy link
Member

@jonas jonas commented Feb 5, 2018

Supersedes #121 with improvements to the sbt 1.x version of the plugin and the documentation.

@jonas
Copy link
Member Author

jonas commented Feb 5, 2018

@rhpvorderman I've rebased your PR and added a few fixes. It would be great if you could take a look since I have never used Laika nor the sbt-site plugin.

I've changed the sbt 1.x version to scope the Laika settings to laikaSite (which should probably be renamed to laika-site) which fixed the path issue. In addition, I've also removed the "hack" to override makeSite and instead set the mappings by calling Laika's laikaSite task.

Ensures that the LaikaSite plugin automatically loads the LaikaPlugin
when enabled. Also reenabled the excludeFilter.
@jonas
Copy link
Member Author

jonas commented Feb 5, 2018

Yay, it’s green. Once I get a confirmation I’ll be happy to cut a release.

@rhpvorderman
Copy link
Contributor

Also ran it on our documentation test suite and it works!
@jonas much much thanks!

@jonas jonas mentioned this pull request Feb 12, 2018
@jonas jonas merged commit a7a59e5 into develop Feb 12, 2018
@jonas jonas deleted the laika-1.0 branch February 12, 2018 02:21
@jonas jonas mentioned this pull request Feb 12, 2018
@jonas jonas added this to the 1.3.2 milestone Feb 12, 2018
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.

2 participants