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.

Sign up for GitHub

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

Jump to bottom

fix warnings #22

Closed
cj opened this issue Jun 10, 2015 · 2 comments
Closed

fix warnings #22

cj opened this issue Jun 10, 2015 · 2 comments

Comments

@cj
Copy link

cj commented Jun 10, 2015

I'll create a pull request for it at some point this week.
@wied03
Copy link
Contributor

wied03 commented Aug 19, 2015

@cj - It's not really easy to eliminate the source of the warnings since RSpec's code itself is what's doing it (we built a decent amount of code to extract all the dynamic requires @ development time). As far as I know, all we can do is tell Opal to ignore the warnings, which may be fine. @elia - You agree?

@wied03
Copy link
Contributor

wied03 commented Oct 20, 2015

Forthcoming 0.5 release suppresses the dynamic require warnings

@wied03 wied03 closed this as completed Oct 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@cj @wied03