Skip to content
This repository has been archived by the owner on Nov 8, 2024. It is now read-only.

Installable also without compiler (Drafter NPM) #51

Merged
merged 1 commit into from
Jul 21, 2016

Conversation

honzajavorek
Copy link
Contributor

New Fury adapter for parsing API Blueprint uses Drafter NPM, which decides whether to use C++ version or JS version of Drafter. This way, Dredd Transactions should become installable also on machines without C++11 compiler.

Also upgraded Swagger adapter to the latest version. No differences expected.

@@ -37,7 +37,7 @@
"jscoverage": "^0.6.0",
"mocha": "^2.3.4",
"mocha-lcov-reporter": "^1.0.0",
"protagonist": "",
Copy link
Member

Choose a reason for hiding this comment

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

You should probably pin to ^1.0.0 since future breaking changes will break your tests.

Copy link
Member

@kylef kylef Jul 21, 2016

Choose a reason for hiding this comment

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

Example, when we drop the blueprint AST we will be releasing 2.0.0 and your tests are using that AST type.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point, thanks!

…ompiler

New Fury adapter for parsing API Blueprint uses Drafter NPM, which decides whether to use C++ version or JS version of Drafter. This way, Dredd Transactions should become installable also on machines without C++11 compiler.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants