Stdlib documentation #339

Mte90 · 2024-07-23T12:29:32Z

Ref: amber-lang/amber-docs#38

I need a bit of help to improve the documentation also because I didn't understood the purpose of some function.

I think that @CymDeveloppement can help me about it as some of them are by him.

src/std/text.ab

src/std/fs.ab

src/std/text.ab

src/std/env.ab

src/std/fs.ab

src/std/text.ab

src/std/env.ab

src/std/array.ab

src/std/env.ab

src/std/text.ab

Co-authored-by: Hubert Jabłoński <[email protected]>

src/std/env.ab

Co-authored-by: Hubert Jabłoński <[email protected]>

we will discuss in a ticket about the error handling

Ph0enixKM

Okay I see now. Let's wait for the PR first

…meantime

I removed the part that create the dependency cycle so we can merge the documentation in the meantime

Mte90 · 2024-08-30T16:12:40Z

So it is ready to get approved :-)

b1ek · 2024-08-31T07:41:09Z

what about the #360? i dont think that has been addressed yet

Mte90 · 2024-08-31T11:03:24Z

Now the PR is free from that issue as I removed the code (the CI works).
In this way we can merge it and do the new release as there is no blockers.
We implemented too many things in this alpha.
When that will be implemented we can improve the stdlib code reusing stuff in the various files.

Ph0enixKM · 2024-09-02T10:10:27Z

I'm still not a big fan of having the len function in a std/text module and nothing like that in std/array module as it works for both text and arrays. How about creating std/core that has the len function and perhaps some more of the most core and atomic functions? @Mte90 @b1ek @KrosFire

Mte90 · 2024-09-02T10:12:37Z

I think that can create confusion.
Usually is a native function in a script language, I think that we can keep as it is now and later move as builtin.

KrosFire · 2024-09-02T18:41:12Z

I'm still not a big fan of having the len function in a std/text module and nothing like that in std/array module as it works for both text and arrays. How about creating std/core that has the len function and perhaps some more of the most core and atomic functions? @Mte90 @b1ek @KrosFire

Let's move it to separate issue. I think that std/core or something like that is a good idea

Ph0enixKM

Looking good

@CymDeveloppement

* feat(doc): stdlib first incomplete version * feat(doc): stdlib first incomplete version * feat(review): by @CymDeveloppement * feat(review): by @CymDeveloppement * Update src/std/array.ab Co-authored-by: Hubert Jabłoński <[email protected]> * Update src/std/text.ab Co-authored-by: Hubert Jabłoński <[email protected]> * Update src/std/text.ab Co-authored-by: Hubert Jabłoński <[email protected]> * Update src/std/text.ab Co-authored-by: Hubert Jabłoński <[email protected]> * Update src/std/env.ab Co-authored-by: Hubert Jabłoński <[email protected]> * Update src/std/env.ab Co-authored-by: Hubert Jabłoński <[email protected]> * Update src/std/fs.ab Co-authored-by: Hubert Jabłoński <[email protected]> * Update src/std/fs.ab Co-authored-by: Hubert Jabłoński <[email protected]> * Update src/std/http.ab Co-authored-by: Hubert Jabłoński <[email protected]> * Update src/std/text.ab Co-authored-by: Hubert Jabłoński <[email protected]> * Update src/std/text.ab Co-authored-by: Hubert Jabłoński <[email protected]> * Update src/std/text.ab Co-authored-by: Hubert Jabłoński <[email protected]> * feat(review): by @KrosFire * feat(review): by @KrosFire * feat(review): text and remove in_array * feat(review): by @CymDeveloppement * fix(std): remove dependence so we can merge the documentation in the meantime * fix(test): wip * fix(test): wip * fix(test): wip * fix(test): wip * fix(test): wip * Update text.ab --------- Co-authored-by: Hubert Jabłoński <[email protected]>

feat(doc): stdlib first incomplete version

4c6f14b

Mte90 added the documentation Improvements or additions to documentation label Jul 23, 2024

Mte90 requested review from CymDeveloppement, Ph0enixKM and KrosFire July 23, 2024 12:29