Skip to content
This repository has been archived by the owner on Aug 14, 2022. It is now read-only.

Remove build process (switch to common.js) #14

Merged
merged 5 commits into from
May 8, 2017
Merged

Remove build process (switch to common.js) #14

merged 5 commits into from
May 8, 2017

Conversation

kaosat-dev
Copy link
Contributor

@kaosat-dev kaosat-dev commented May 6, 2017 •
edited
Loading

This does the following :

  • no more build process
  • less dependencies

Also:

  • flagged some failing tests as expected to fail
  • fixed a few tests/implementation issues

@kaosat-dev
refactor():
f427892 
- switch back to common.js (bye build step)
- enable union between 2d & 3d shapes
- flagged currently failing tests as expected to fail (no obvious fixes/ not implemented etc)
@kaosat-dev
refactor(): migrated to commonjs of forgotten module
dd4808e
@kaosat-dev
chore(cleanup):
f3696ac 
- removed all rollup , babel etc build steps
@kaosat-dev kaosat-dev requested a review from z3dev May 6, 2017 21:49
@kaosat-dev
chore(CI): limiting tests to es6 capable node.js versions
007e791
@kaosat-dev
test(primitives3d): added almost equal polygon compare
5d22183 
- added helper function
- added use of helper function for tests that fail on CI because of precision
z3dev
z3dev approved these changes May 8, 2017
View reviewed changes
Copy link
Member

@z3dev z3dev left a comment

Choose a reason for hiding this comment

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

I didn't see anything wrong with the changes. Looks good.

@z3dev z3dev merged commit 276dce4 into master May 8, 2017
@kaosat-dev kaosat-dev deleted the simplify branch May 23, 2017 07:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@z3dev z3dev z3dev 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.

None yet
2 participants
@kaosat-dev @z3dev