Skip to content

Tenyaeva#18

Open
b08lsoai wants to merge 73 commits into
Kakadu:masterfrom
b08lsoai:develop
Open

Tenyaeva#18
b08lsoai wants to merge 73 commits into
Kakadu:masterfrom
b08lsoai:develop

Conversation

@b08lsoai
Copy link
Copy Markdown

implementation of the miniML interpreter

Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Zanuda-linter report

Comment thread Tenyaeva/tests/generators.ml
Comment thread Tenyaeva/bin/REPL.ml Outdated
Comment thread Tenyaeva/bin/REPL.ml Outdated
Comment thread Tenyaeva/lib/monad.ml Outdated
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Zanuda-linter report

Comment thread Tenyaeva/tests/qcheck.ml
@@ -0,0 +1,32 @@
(** Copyright 2025, Tenyaeva Ekaterina *)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

File 'Tenyaeva/tests/qcheck.ml' doesn't have corresponding .mli interface

Comment thread Tenyaeva/bin/REPL.ml Outdated
open Tenyaeva_lib.Ast

type options =
{ mutable dump_parsetree : bool
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Using mutable data structures for teaching purposes is usually discouraged. Replace Hashtables by standard tree-like maps or consider Hash-Array Mapped Tries (HAMT). Use mutable references and mutable structure fields only if it is really required. In all places where it is needed indeed, describe in a comment why it is needed there.

Comment thread Tenyaeva/bin/REPL.ml Outdated

type options =
{ mutable dump_parsetree : bool
; mutable max_steps : int
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Using mutable data structures for teaching purposes is usually discouraged. Replace Hashtables by standard tree-like maps or consider Hash-Array Mapped Tries (HAMT). Use mutable references and mutable structure fields only if it is really required. In all places where it is needed indeed, describe in a comment why it is needed there.

Comment thread Tenyaeva/lib/monad.ml Outdated
let get st = st, Ok st
let put new_st _st = new_st, Ok ()
let fail e st = st, Error e
let run m st = m st
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Eta reduction proposed. It's recommended to rewrite 'fun st -> m st' as
'm'

Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Zanuda-linter report

Comment thread Tenyaeva/lib/monad.ml
Comment thread Tenyaeva/bin/REPL.ml
open Tenyaeva_lib.Ast

type options =
{ mutable dump_parsetree : bool (* changed via command line argument *)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Using mutable data structures for teaching purposes is usually discouraged. Replace Hashtables by standard tree-like maps or consider Hash-Array Mapped Tries (HAMT). Use mutable references and mutable structure fields only if it is really required. In all places where it is needed indeed, describe in a comment why it is needed there.

Comment thread Tenyaeva/bin/REPL.ml
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Zanuda-linter report

Comment thread Tenyaeva/lib/monad.ml
Comment thread Tenyaeva/bin/REPL.ml
Comment thread Tenyaeva/bin/REPL.ml

type options =
{ mutable dump_parsetree : bool (* changed via command line argument *)
; mutable max_steps : int (* changed via command line argument *)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Using mutable data structures for teaching purposes is usually discouraged. Replace Hashtables by standard tree-like maps or consider Hash-Array Mapped Tries (HAMT). Use mutable references and mutable structure fields only if it is really required. In all places where it is needed indeed, describe in a comment why it is needed there.

Signed-off-by: Tenyaeva Ekaterina <b08lsoai@gmail.com>
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 3, 2026

Документация и тестовое покрытие (98.53%) должны скоро появиться.

https://kakadu.github.io/fp25/docs/Tenyaeva

https://kakadu.github.io/fp25/cov/Tenyaeva

2026-03-03 20:02

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Zanuda-linter report

Comment thread Tenyaeva/bin/REPL.ml
open Tenyaeva_lib.Ast

type options =
{ mutable dump_parsetree : bool (* changed via command line argument *)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Using mutable data structures for teaching purposes is usually discouraged. Replace Hashtables by standard tree-like maps or consider Hash-Array Mapped Tries (HAMT). Use mutable references and mutable structure fields only if it is really required. In all places where it is needed indeed, describe in a comment why it is needed there.

Comment thread Tenyaeva/bin/REPL.ml

type options =
{ mutable dump_parsetree : bool (* changed via command line argument *)
; mutable max_steps : int (* changed via command line argument *)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Using mutable data structures for teaching purposes is usually discouraged. Replace Hashtables by standard tree-like maps or consider Hash-Array Mapped Tries (HAMT). Use mutable references and mutable structure fields only if it is really required. In all places where it is needed indeed, describe in a comment why it is needed there.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 3, 2026

Linter report from 2026-03-03 20:03, for mini language Tenyaeva

File "bin/REPL.ml", lines 10-11, characters 4-3:
10 | ....mutable dump_parsetree : bool (* changed via command line argument *)
11 |   ;................................................................
Alert zanuda-linter: Using mutable data structures for teaching purposes is usually discouraged. Replace Hashtables by standard tree-like maps or consider Hash-Array Mapped Tries (HAMT). Use mutable `ref`erences and mutable structure fields only if it is really required. In all places where it is needed indeed, describe in a comment why it is needed there.
File "bin/REPL.ml", line 11, characters 4-27:
11 |   ; mutable max_steps : int (* changed via command line argument *)
         ^^^^^^^^^^^^^^^^^^^^^^^
Alert zanuda-linter: Using mutable data structures for teaching purposes is usually discouraged. Replace Hashtables by standard tree-like maps or consider Hash-Array Mapped Tries (HAMT). Use mutable `ref`erences and mutable structure fields only if it is really required. In all places where it is needed indeed, describe in a comment why it is needed there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant