Reviewed all commit messages.
Reviewable status: 0 of 3 files reviewed, 1 unresolved discussion (waiting on @Gerson2102)

crates/cairo-lang-semantic/src/inline_macros/array.rs line 90 at r1 (raw file):

            "#}
            .to_string(),

remove whitespace lines, and shorten lines longer than 100 chars.

Code quote:

    fn documentation(&self) -> Option<String> {
        Some(
            indoc! {r#"
            Creates a new array containing the provided elements.

            The `array!` macro allows you to create an array by specifying a list of elements. \
            The elements are added to a new array in the order they are provided.

            # Syntax

            ```cairo
            array![element1, element2, element3, ...]
            ```

            # Returns

            An array containing the specified elements.

            # Examples

            ```cairo
            let arr = array![]; // Creates an empty array.

            let arr = array![1, 2, 3]; // Creates an array containing 1, 2, and 3.

            let x = 5;
            let y = 10;
            let arr = array![x, y, x + y]; // Creates an array containing 5, 10, and 15.
            ```

            # Notes

            - All elements must be of the same type or compatible types that can be coerced to a common type.
            - The macro internally uses `ArrayTrait::new()` to create a new array and `ArrayTrait::append()` to add each element.
            - The order of the elements in the array is the same as the order they are provided in the macro.

            "#}
            .to_string(),

Reviewed 1 of 3 files at r2.
Reviewable status: 1 of 3 files reviewed, 2 unresolved discussions (waiting on @Gerson2102)

crates/cairo-lang-semantic/src/inline_macros/assert.rs line 11 at r2 (raw file):

use cairo_lang_syntax::node::ast::WrappedArgList;
use cairo_lang_syntax::node::db::SyntaxGroup;
use cairo_lang_syntax::node::{ast, TypedStablePtr, TypedSyntaxNode};

Please rebase to use the latest rust fmt version.

Reviewable status: 1 of 3 files reviewed, 2 unresolved discussions (waiting on @Gerson2102 and @orizi)

a discussion (no related file):

1. Please use Reviewable for commenting

I dont know why I'm getting these errors for some of the macros related to format. Look these screenshots:

You need to import the indoc macro, add use indoc::indoc

Reviewed all commit messages.
Reviewable status: 1 of 3 files reviewed, 2 unresolved discussions (waiting on @Gerson2102 and @mkaput)

crates/cairo-lang-semantic/src/inline_macros/array.rs line 80 at r2 (raw file):

            - Elements are added in the order provided.
            "#}
            .to_string(),

Suggestion:

    fn documentation(&self) -> Option<String> {
        Some(
            indoc! {r#"
            Creates a new array containing the provided elements.
            The `array!` macro allows you to create an array by specifying a list of elements. \
            The elements are added to a new array in the order they are provided.

            # Syntax
            ```cairo
            array![element1, element2, element3, ...]
            ```
            # Returns
            An array containing the specified elements.

            # Examples
            ```cairo
            let arr = array![]; // Creates an empty array.
            let arr = array![1, 2, 3]; // Creates an array containing 1, 2, and 3.
            let x = 5;
            let y = 10;
            let arr = array![x, y, x + y]; // Creates an array containing 5, 10, and 15.
            ```
            # Notes
            - Elements must be of the same type or convertible to a common type.
            - Uses `ArrayTrait::new()` to create the array and `ArrayTrait::append()` to add \
            elements.
            - Elements are added in the order provided.
            "#}
            .to_string(),

Reviewed 1 of 2 files at r3, all commit messages.
Reviewable status: 1 of 3 files reviewed, 3 unresolved discussions (waiting on @Gerson2102 and @mkaput)

crates/cairo-lang-semantic/src/inline_macros/assert.rs line 11 at r2 (raw file):

now also run ./scripts/rust_fmt.sh before pushing.

this should revert the bad formatting changes.

crates/cairo-lang-semantic/src/inline_macros/assert.rs line 132 at r3 (raw file):

            - `condition`: A boolean expression to evaluate.
            - `format_string` (optional): A string literal for format placeholders.
            - `args` (optional): Arguments corresponding to the format placeholders in the `format_string`.

shorten or split line. (under 100 rust chars)
also elsewhere if still exists.

Code quote:

            - `args` (optional): Arguments corresponding to the format placeholders in the `format_string`.

crates/cairo-lang-semantic/src/inline_macros/assert.rs line 142 at r3 (raw file):

            // Panics with "Age must be at least 21, found 18."
            let x = -1;
            assert!(x >= 0, "Invalid value: x = {}", x);

maybe we should have an example with this style?
in panic as well.

Suggestion:

            assert!(x >= 0, "Invalid value: x = {x}");

Reviewed 3 of 3 files at r4, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @Gerson2102)

Reviewed all commit messages.
Reviewable status: 3 of 4 files reviewed, 2 unresolved discussions (waiting on @Gerson2102 and @mkaput)

crates/cairo-lang-semantic/src/inline_macros/consteval_int.rs line 69 at r5 (raw file):

        Some(
            indoc! {r#"
            Evaluates an integer expression at compile time.

State the fact or is deprecated, as can be done with normal consts.

crates/cairo-lang-semantic/src/inline_macros/consteval_int.rs line 86 at r5 (raw file):

            ```
            # Notes
            - Useful for compile-time computations when const expressions were not fully supported.

It no longer is useful.

It is in the test plugin, and not the general plugin

Reviewable status: 3 of 4 files reviewed, 2 unresolved discussions (waiting on @Gerson2102 and @mkaput)

Reviewed 1 of 1 files at r5, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @Gerson2102)

Reviewed 1 of 1 files at r6, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @Gerson2102)

Reviewed all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @Gerson2102)

Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @Gerson2102)