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

Clean up documentation for x-api v3 #12

Merged
merged 2 commits into from
Nov 26, 2015
Merged

Clean up documentation for x-api v3 #12

merged 2 commits into from
Nov 26, 2015

Conversation

kytrinyx
Copy link
Member

This is in support of ditching the help site and integrating the docs into the main site's language-specific pages (served via the same API that serves the exercise data).

@kytrinyx
Copy link
Member Author

The tests are failing, but rerunning the tests on master also makes them fail, which is odd, since those tests used to pass. Not sure what's going on.

@yurrriq
Copy link
Member

yurrriq commented Nov 25, 2015

Looks like the version of LFE getting pulled is using rebar3, so I'll have to update everything accordingly. As such, I've opened #13.

@kytrinyx
Copy link
Member Author

Ok, thanks!

The tests are broken and LFE supports rebar3 now, which is great.
In order to get xlfe working with rebar3, I had to update all the
`Makefile`s and `rebar.config`s. The deps symlinks to point to ../_build
and the test make target uses _build/default/lib/... instead of deps/...

Now the tests pass! 🎉

- Update .travis.yml
  - Can't run rebar3 without downloading it first...
  - Add PWD to PATH so make can find rebar3.

- Update test script
  - ({"init terminating in do_boot",
     {badarg, [{erl_prim_loader,check_file_result,3,[]},
     {init, get_boot, 1, []},
     ... ]})
    make: *** [test] User defined signal 2
  - 💋 and use maps

- Update config.json
  - Ignore _build
yurrriq added a commit that referenced this pull request Nov 26, 2015
Clean up documentation for x-api v3
@yurrriq yurrriq merged commit 9a69b46 into master Nov 26, 2015
@kytrinyx kytrinyx deleted the docs branch November 27, 2015 01:30
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