Skip to content

Verbose#7700

Closed
Omartech312 wants to merge 2 commits intoRdatatable:masterfrom
kkarissa:Verbose
Closed

Verbose#7700
Omartech312 wants to merge 2 commits intoRdatatable:masterfrom
kkarissa:Verbose

Conversation

@Omartech312
Copy link
Copy Markdown

  • name: rfunStr

title: rfunStr

type: string

description: specifies function name

constraints: 6-character word limit, only used when verbose is true.

@Omartech312 Omartech312 requested a review from jangorecki as a code owner April 7, 2026 00:01
@MichaelChirico
Copy link
Copy Markdown
Member

AI tooling is way too good by now to be signing your name to slop like this and wasting reviewers' time

@jangorecki
Copy link
Copy Markdown
Member

@igorsteinmacher
Copy link
Copy Markdown
Contributor

Hi @MichaelChirico and @jangorecki, @Omartech312 is a student in our Open source course at NAU, and they actually worked on the problem.

I can tell that they did not use AI; they worked in class on this. Is there anything that led you to flag this as "AI"? Would you let they know what mistake they actually made to be banned?

Thanks a lot.

@jangorecki
Copy link
Copy Markdown
Member

jangorecki commented Apr 21, 2026

Post is not informative at all, not even a title, improperly formatted.
Changes to code does not seem to be robust - we can keep list of function on the side, no need to keep branching with if verbose so many times. There are changes which are not related to the matter of the PR - extra brackets which doesn't change any logic.

For people who uses GH for the first time I would suggest to try PR against their own fork to learn the process.

@igorsteinmacher
Copy link
Copy Markdown
Contributor

Thanks @jangorecki . That's exactly what I told them: to work on a fork from the fork. Thanks for clarifying. Would it be possible to unblock the student?

@MichaelChirico
Copy link
Copy Markdown
Member

OK, unblocked

@Omartech312
Copy link
Copy Markdown
Author

We apologize for this pull request. I meant to upload this file to our forked repository (hence why the information was incomplete and not formatted properly), not to the main repo, so that my teammates have access to it and test it on their local systems. From now on, we will make sure to have a fork of our fork.

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.

4 participants