Skip to content

more math functions - #5

Open
js6450 wants to merge 3 commits into
Open-Source-Studio-at-ITP:masterfrom
js6450:master
Open

more math functions#5
js6450 wants to merge 3 commits into
Open-Source-Studio-at-ITP:masterfrom
js6450:master

Conversation

@js6450

@js6450 js6450 commented Oct 1, 2018

Copy link
Copy Markdown

added =randInt, arrayLen, factorize and 3D distance calc functions and its tests

@shiffman

shiffman commented Oct 1, 2018

Copy link
Copy Markdown
Member

Hopefully I resolved the merge conflicts correctly! @NHibiki @js6450 feel free to check my work! If the tests pass, I'll merge 😄

@shiffman

shiffman commented Oct 1, 2018

Copy link
Copy Markdown
Member

Ooops.

Comment thread .idea/UnitTestingCI.iml
@@ -0,0 +1,12 @@
<?xml version="1.0" encoding="UTF-8"?>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What are these files that are added to .idea directory?

Comment thread package.json Outdated
},
"repository": {
"type": "git",
"url": "git+https://github.com/js6450/UnitTestingCI.git"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Maybe these URLs should point to Open-Source-Studio-at-ITP rather than js6450?

Change from js6450 -> Open-Source-Studio-at-ITP/UnitTestingCI
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.

2 participants