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

[ SI-7514 ] Introduce Source.fromClassPath(resource) method #4538

Closed
wants to merge 1 commit into from
Closed

[ SI-7514 ] Introduce Source.fromClassPath(resource) method #4538

wants to merge 1 commit into from

Conversation

jdevelop
Copy link
Contributor

@jdevelop jdevelop commented Jun 4, 2015

Returns Source from named classpath resource
Simplifies

val src = io.Source.fromInputStream(classOf[ClassInst].getResourceAsStream("/name"))

to

val src = io.Source.fromClassPath("/name")

@scala-jenkins scala-jenkins added this to the 2.11.7 milestone Jun 4, 2015
Creates Source from named classpath resource.

Simplifies

 val src = io.Source.fromInputStream(classOf[ClassInst].getResourceAsStream("/name"))

to

 val src = io.Source.fromClassPath("/name")
@SethTisue
Copy link
Member

I like it, but this would need to target 2.12, not 2.11

@SethTisue SethTisue closed this Jun 17, 2015
@jdevelop
Copy link
Contributor Author

@SethTisue do I need to do something about it? Thanks!

@SethTisue
Copy link
Member

GitHub doesn't allow retargeting an existing PR to a new branch, so a new PR is needed

@jdevelop
Copy link
Contributor Author

@SethTisue ok, thanks! Done.

#4561

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.

3 participants