Closed JamesCarlyleClarke closed 1 month ago
For the methods, I wrote the documentation in a way that I think is more informative about the functions than the documentation you gave here. Fair point about the constants, though - I'll add documentation for those.
...I feel like I missed the point of what you're saying. Did you just mean "the documentation isn't worded the same", or were you trying to say something else?
Well, seems like I missed you updating the ReadMe to mention index.d.ts so my point is now partially lost, sorry! And by the way, if my past point or coming comments sounds like criticism then sorry, it really is not meant that way, it's a great library.
Still: 1) From the readme, [break_eternity.js] 'has the same interface, so it can be used as a drop-in replacement for break_infinity.js and decimal.js'. But a) the constants aren't the same, so it's not going to work as a 'drop-in', and b), I was using https://patashu.github.io/break_infinity.js/classes/default.html as documentation since it's got the same interface and is a drop-in replacement, so the constants didn't match there either.
2) 'A list of functions is provided earlier in this readme' but unless I'm being really stupid, I couldn't see one. And again, I had not noticed the new index.d.ts.
So apropos point 2 I thought perhaps that list could be provided for the readme, which is why I gave the list. I figured either it could be used in some simple list form, or as a basis for transferring the break_infinity.js docs. As it turns out, you have done something with index.d.ts so it's not so important, though maybe a simpler 'readme' form would be good in addition to the index?
Hope that helps.
v2.1.0 has added documentation for the constants.
I'm still not sure if I actually covered your whole issue, so I'm not going to close it myself. Feel free to close the issue if the documentation for the constants is sufficient for what you were looking for.
Yeah, I took it to mean 'like-for-like' replacement where the methods, constants etc. match except in the few cases specified as not matching.
Anyway, there's documentation, I'm a happy bunny (and sorry for missing the update before I posted). Calling it closed, have a great one!
Anyway, there's documentation, I'm a happy bunny (and sorry for missing the update before I posted). Calling it closed, have a great one!
You didn't "miss the update before you posted", I just released the update an hour ago.
No, I meant not seeing the change to the ReadMe where you added a reference to index.d.ts, thus giving info on where we could find the methods, before I posted the original issue. I read it a few months back, when it just mentioned the ReadMe and not index.d.ts, and I never spotted the change.
Some of the constants don't match the break_infinity.js documentation; I'm not sure about the methods. I produced an automated list, I have not manually checked this, but it might be a useful start for any additional documentation.
Here are the constants:
Here is the list of functions, with their parameters and brief explanations: