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

Change zinc to support tests #49

Merged
merged 1 commit into from May 31, 2014

Conversation

Projects
None yet
2 participants
@bharrisau
Contributor

bharrisau commented May 31, 2014

Adds the required cfg attributes to build with --test and mocks out
all the functions with inline asm.

Change zinc to support tests
Adds the required cfg attributes to build with --test and mocks out
all the functions with inline asm.

  - [ ] Get zinc to build with test
  - [ ] Add test to Rakefile and travis
  - [ ] Refactor inline asm (move to designated area, dummy with `fail!`
@bharrisau

This comment has been minimized.

Contributor

bharrisau commented May 31, 2014

@farcaller If you'd rather me not destroy your Rakefile, feel free to take the branch and finish it off. The tests build and run so I'm happy.

@farcaller

This comment has been minimized.

Member

farcaller commented May 31, 2014

LGTM. Checklist moved to #50.

farcaller added a commit that referenced this pull request May 31, 2014

Merge pull request #49 from bharrisau/first-test
Change zinc to support tests

@farcaller farcaller merged commit fe2f3ec into hackndev:master May 31, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment