Skip to content

qmlweb_jsparse: Include pragma statements #26

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
Merged
merged 1 commit into from
Nov 14, 2016
Merged

qmlweb_jsparse: Include pragma statements #26

merged 1 commit into from
Nov 14, 2016

Conversation

stephenmdangelo
Copy link
Member

No description provided.

@stephenmdangelo stephenmdangelo force-pushed the include_pragma_statements branch from 9d9bea5 to a71fece Compare November 11, 2016 19:37
ChALkeR
ChALkeR approved these changes Nov 12, 2016
View reviewed changes
Copy link
Member

@ChALkeR ChALkeR left a comment

Choose a reason for hiding this comment

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

LGTM.

Not sure why the file endings changed (looks like the tooling here doesn't generate newlines at the end of files), but that doesn't matter much.

@stephenmdangelo
qmlweb_jsparse: Include pragma statements
44c339f 
PR-URL: qmlweb#26
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
@stephenmdangelo stephenmdangelo force-pushed the include_pragma_statements branch from a71fece to 44c339f Compare November 14, 2016 16:07
@stephenmdangelo stephenmdangelo merged commit a640359 into qmlweb:master Nov 14, 2016
@ChALkeR ChALkeR mentioned this pull request Dec 26, 2016
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ChALkeR ChALkeR ChALkeR approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@stephenmdangelo @ChALkeR