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

fix #41 test-08-failOnClassesExcludedFromJarStar #43

Closed

Conversation

ktoso
Copy link
Member

@ktoso ktoso commented Aug 15, 2017

Resolves #41

The replacing was too aggressive leading to wrong things happening; we just want to drop the trailing .* if present

@ktoso
Copy link
Member Author

ktoso commented Aug 15, 2017

FYI @xuwei-k

@ktoso
Copy link
Member Author

ktoso commented Aug 15, 2017

I'm confused about what's going on here actually now... will look into it more later

.travis.yml Outdated
@@ -1,4 +1,5 @@
language: scala
sudo: false
script: sbt scripted
Copy link
Contributor

Choose a reason for hiding this comment

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

There are also test cases that need to be run, so sbt test is required.

@larsrh
Copy link
Contributor

larsrh commented Aug 15, 2017

I'm confused about what's going on here actually now... will look into it more later

@ktoso pretty sure that the -> in the failing test needs to be replaced by >.

@ktoso
Copy link
Member Author

ktoso commented Feb 13, 2018

I missed your reply here, sorry about that @larsrh ! Will try then!
And release a new version today I think

@larsrh
Copy link
Contributor

larsrh commented Feb 13, 2018

I missed your reply here, sorry about that @larsrh ! Will try then!

No worries, I have no recollection of even posting that comment 😆

@ktoso
Copy link
Member Author

ktoso commented Feb 13, 2018

Yeah I'd also rather not remember any work on anything osgi related ;)

@ktoso
Copy link
Member Author

ktoso commented Feb 13, 2018

I think it SHOULD fail then... leaving this open for now

@ktoso ktoso closed this Jan 23, 2021
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.

test-08-failOnClassesExcludedFromJarStar fail
3 participants