Skip to content

Resolves #1220, Implement File.read in Nashorn#1222

Merged
elia merged 2 commits intoopal:masterfrom
ggrossetie:issue-1220
Dec 6, 2015
Merged

Resolves #1220, Implement File.read in Nashorn#1222
elia merged 2 commits intoopal:masterfrom
ggrossetie:issue-1220

Conversation

@ggrossetie
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can you remove the < IO part just to avoid conflicts if the superclass of File ever changes?

elia added a commit that referenced this pull request Dec 6, 2015
Resolves #1220, Implement File.read in Nashorn
@elia elia merged commit f58cf3e into opal:master Dec 6, 2015
@elia
Copy link
Copy Markdown
Member

elia commented Dec 6, 2015

Thanks, I think we should tweak the pattern a bit and maybe move the platform detection to corelib/constants and have opal/platform just case OPAL_PLATFORM. Also I was unsure about having the implementation for File required automatically but I think it makes sense for most use cases.

@ggrossetie
Copy link
Copy Markdown
Member Author

You are welcome 😄

Also I was unsure about having the implementation for File required automatically but I think it makes sense for most use cases.

What is the drawback ? A bigger Javascript file ?

@ggrossetie ggrossetie deleted the issue-1220 branch April 11, 2021 15:04
hmdne pushed a commit to hmdne/opal that referenced this pull request Jan 27, 2024
Bumps [pug](https://github.com/pugjs/pug) from 3.0.0 to 3.0.2.
- [Release notes](https://github.com/pugjs/pug/releases)
- [Commits](https://github.com/pugjs/pug/compare/pug@3.0.0...pug@3.0.2)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
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