Skip to content

Rayanouerdane/det 21 ci that automatically adds comments to the pull request - #39

Open
Rayan-ouer wants to merge 63 commits into
mainfrom
rayanouerdane/det-21-ci-that-automatically-adds-comments-to-the-pull-request
Open

Rayanouerdane/det 21 ci that automatically adds comments to the pull request#39
Rayan-ouer wants to merge 63 commits into
mainfrom
rayanouerdane/det-21-ci-that-automatically-adds-comments-to-the-pull-request

Conversation

@Rayan-ouer

Copy link
Copy Markdown
Collaborator

Description

Please provide a detailed description of what was done in this PR.
Precise the issue that you are resolving.

Changes include

  • Bugfix (non-breaking change that solves an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (change that is not backwards-compatible and/or changes current functionality)

Breaking changes

Please complete this section if any breaking changes have been made, otherwise delete it.

Checklist

  • I have assigned this PR to myself
  • I have added at least 1 reviewer
  • I have added the needed labels
  • I have linked this PR to an issue
  • I have linked this PR to a milestone
  • I have linked this PR to a project
  • I have tested this code
  • I have added / updated tests (unit / functional / end-to-end / ...)
  • I have updated the README and other relevant documents (guides...)
  • I have added sufficient documentation both in code, as well as in the READMEs

Additional comments

Please post additional comments in this section if you have them, otherwise delete it.

@linear-code

linear-code Bot commented Jul 14, 2026

Copy link
Copy Markdown

DET-21

@Rayan-ouer Rayan-ouer closed this Jul 21, 2026
@Rayan-ouer Rayan-ouer reopened this Jul 21, 2026
@Rayan-ouer Rayan-ouer closed this Jul 21, 2026
@Rayan-ouer Rayan-ouer reopened this Jul 21, 2026
@Rayan-ouer Rayan-ouer closed this Jul 21, 2026
@Rayan-ouer Rayan-ouer reopened this Jul 21, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Debtlint suggestion

This code can be refactorised.

                    start: Position {
                        line: 10,
                        character: 1,
                    },
                    end: Position {
                        line: 19,
                        character: 1,
                    },
                },
    let files = vec![SourceFile {
        path: args.file.clone(),
        content: content.clone(),
    }];
    let result = run_bpe(
        &files,

@github-actions

Copy link
Copy Markdown
Contributor

Debtlint suggestion

This code is duplicated.

        VocabularyEntry::Symbol(ch) => ch.to_string(), // if the entry is a symbol return the string
        VocabularyEntry::Merge { pair, .. } => {
            // if the entry is a merge return the pair
            let (left, right) = *pair; // destructure the pair
            let mut text = decode_token(left, vocabulary); // decode the left token
            text.push_str(&decode_token(right, vocabulary)); // decode the right token and add to the string
            text
        }
    } // return the string
    let files = vec![SourceFile {
        path: args.file.clone(),
        content: content.clone(),
    }];
    let result = run_bpe(
        &files,

@PoCInnovation PoCInnovation deleted a comment from github-actions Bot Aug 11, 2026
@PoCInnovation PoCInnovation deleted a comment from github-actions Bot Aug 11, 2026
@PoCInnovation PoCInnovation deleted a comment from github-actions Bot Aug 11, 2026
@PoCInnovation PoCInnovation deleted a comment from github-actions Bot Aug 11, 2026
@PoCInnovation PoCInnovation deleted a comment from github-actions Bot Aug 11, 2026
@PoCInnovation PoCInnovation deleted a comment from github-actions Bot Aug 11, 2026
@PoCInnovation PoCInnovation deleted a comment from github-actions Bot Aug 11, 2026
@PoCInnovation PoCInnovation deleted a comment from github-actions Bot Aug 11, 2026
@PoCInnovation PoCInnovation deleted a comment from github-actions Bot Aug 11, 2026
@PoCInnovation PoCInnovation deleted a comment from github-actions Bot Aug 11, 2026
@PoCInnovation PoCInnovation deleted a comment from github-actions Bot Aug 11, 2026
@PoCInnovation PoCInnovation deleted a comment from github-actions Bot Aug 11, 2026
@PoCInnovation PoCInnovation deleted a comment from github-actions Bot Aug 11, 2026
@PoCInnovation PoCInnovation deleted a comment from github-actions Bot Aug 11, 2026
@PoCInnovation PoCInnovation deleted a comment from github-actions Bot Aug 11, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Debtlint suggestion

This code can be refactorised.

                    start: Position {
                        line: 10,
                        character: 1,
                    },
                    end: Position {
                        line: 19,
                        character: 1,
                    },
                },
    let files = vec![SourceFile {
        path: args.file.clone(),
        content: content.clone(),
    }];
    let result = run_bpe(
        &files,

@github-actions

Copy link
Copy Markdown
Contributor

Debtlint suggestion

This code is duplicated.

        VocabularyEntry::Symbol(ch) => ch.to_string(), // if the entry is a symbol return the string
        VocabularyEntry::Merge { pair, .. } => {
            // if the entry is a merge return the pair
            let (left, right) = *pair; // destructure the pair
            let mut text = decode_token(left, vocabulary); // decode the left token
            text.push_str(&decode_token(right, vocabulary)); // decode the right token and add to the string
            text
        }
    } // return the string
    let files = vec![SourceFile {
        path: args.file.clone(),
        content: content.clone(),
    }];
    let result = run_bpe(
        &files,

@github-actions

Copy link
Copy Markdown
Contributor

Debtlint suggestion

This code can be refactorised.

                    start: Position {
                        line: 10,
                        character: 1,
                    },
                    end: Position {
                        line: 19,
                        character: 1,
                    },
                },
    let files = vec![SourceFile {
        path: args.file.clone(),
        content: content.clone(),
    }];
    let result = run_bpe(
        &files,

@github-actions

Copy link
Copy Markdown
Contributor

Debtlint suggestion

This code is duplicated.

        VocabularyEntry::Symbol(ch) => ch.to_string(), // if the entry is a symbol return the string
        VocabularyEntry::Merge { pair, .. } => {
            // if the entry is a merge return the pair
            let (left, right) = *pair; // destructure the pair
            let mut text = decode_token(left, vocabulary); // decode the left token
            text.push_str(&decode_token(right, vocabulary)); // decode the right token and add to the string
            text
        }
    } // return the string
    let files = vec![SourceFile {
        path: args.file.clone(),
        content: content.clone(),
    }];
    let result = run_bpe(
        &files,

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.

1 participant