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

provide possibility to set up data dir #262

Closed
wants to merge 2 commits into from

Conversation

btomala
Copy link

@btomala btomala commented Sep 18, 2018

The changes which I requested in issue #261

I hope it will be helpful.

@btomala
Copy link
Author

btomala commented Sep 18, 2018

This should solve the issue #224 (comment) by the setting up data directory.

@btomala
Copy link
Author

btomala commented Sep 21, 2018

@gslowikowski I will glad if you can consider merging this feature as well.

@gregghz
Copy link

gregghz commented Dec 13, 2019

Will this ever be merged?

@zabbal
Copy link

zabbal commented Mar 31, 2020

I'm interested as well - what's blocking the merge?

@@ -77,7 +78,7 @@ object ScoverageSbtPlugin extends AutoPlugin {
}
Seq(
Some(s"-Xplugin:${pluginPath.getAbsolutePath}"),
Some(s"-P:scoverage:dataDir:${crossTarget.value.getAbsolutePath}/scoverage-data"),
Some(s"-P:scoverage:dataDir:${coverageDataDir.value.getAbsolutePath}/scoverage-data"),

Choose a reason for hiding this comment

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

Probably remove the /scoverage-data part, as this can be set by the dataDir

@ckipp01
Copy link
Member

ckipp01 commented May 7, 2021

Hello! If there is still an interest in getting this merged, feel free to rebase and I can take a look at this.

@btomala
Copy link
Author

btomala commented May 7, 2021

If someone is interested in this can take it over.

@pzhong-livongo
Copy link
Contributor

I just created a new PR where it's rebasing the latest master.

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.

6 participants