Skip to content

Pull Request to Master Branch #55

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

Closed

Conversation

erichilarysmithsr
Copy link

Pull request to Master Branch for updating index.html

New index html page for Netlify
Updated index.html page for Netlify Dev
Updated index.html with a form from netlify
Add mathjs server.js file at the root
Add math workerjs file for the browser support in HTML and JS
adding requirejs file
@infusion infusion closed this Oct 27, 2022
@Rudxain
Copy link

Rudxain commented Oct 29, 2022

wtf is this? This is totally off-topic

@erichilarysmithsr
Copy link
Author

erichilarysmithsr commented Oct 29, 2022 via email

@Rudxain
Copy link

Rudxain commented Oct 31, 2022

  • This is not a website, it's a library (which does have a website, but for docs)
  • This isn't NodeJS-specific
  • This is not "Basic Math Skills", and this lib isn't made only for learning purposes (it's a lib that adds fraction objects, to do rational arithmetic)

@erichilarysmithsr
Copy link
Author

erichilarysmithsr commented Oct 31, 2022 via email

@Rudxain
Copy link

Rudxain commented Nov 1, 2022

I know how to use the math library

I didn't meant to say that you don't, I was just describing the lib. I'm sorry if I sounded harsh

it's reusable code

That's true, you're right

Support it for educational purposes

What do you mean? Is that "I support it" or "you should support it"? Either way, that's nice! I agree

I meant that even though it can be used as educational resource, that's not the main purpose

@erichilarysmithsr
Copy link
Author

erichilarysmithsr commented Nov 1, 2022 via email

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.

3 participants