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

TIMOB-10215 If JSCA file creation contains runtime errors, fail it immed... #2688

Merged
merged 1 commit into from Aug 9, 2012
Merged

TIMOB-10215 If JSCA file creation contains runtime errors, fail it immed... #2688

merged 1 commit into from Aug 9, 2012

Conversation

billdawson
Copy link
Contributor

...iately so that our scons build will also failed. This avoids accidentally creating releases which contain bad or partial api.jsca files.

See JIRA:

https://jira.appcelerator.org/browse/TIMOB-10215

…mediately so that our scons build will also failed. This avoids accidentally creating releases which contain bad or partial api.jsca files.
@arthurevans
Copy link
Contributor

Reviewed code and tested. Verified with the fix installed, the build fails if there's an error in the doc syntax, and succeeds if there are no doc errors.

Without the fix installed, the build succeeds whether or not the JSCA build throws an error.

Accepted.

Note that master currently contains a syntax error in TextArea.yml that will break the build. I'll check in a fix for that separately.

@arthurevans
Copy link
Contributor

Doc fix to ensure that the build doesn't break:

#2705

arthurevans pushed a commit that referenced this pull request Aug 9, 2012
TIMOB-10215 If JSCA file creation contains runtime errors, fail it immed...
@arthurevans arthurevans merged commit e131da2 into tidev:master Aug 9, 2012
@billdawson
Copy link
Contributor Author

#2707 is a 2_1_X cherry-pick of this.

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.

None yet

2 participants