Skip to content

Fix subtitle patch#139

Open
chandrashekarazad-ext wants to merge 1 commit intoasticode:masterfrom
chandrashekarazad-ext:fix/subtitle-patch
Open

Fix subtitle patch#139
chandrashekarazad-ext wants to merge 1 commit intoasticode:masterfrom
chandrashekarazad-ext:fix/subtitle-patch

Conversation

@chandrashekarazad-ext
Copy link

No description provided.

@asticode
Copy link
Owner

Hi, has this been generated by AI?

@chandrashekarazad-ext
Copy link
Author

Yes, but validated

@dlecorfec
Copy link
Contributor

Hi @chandrashekarazad-ext, nothing wrong with AI, but:

  • you didn't provide a clear description of the PR and justification for the changes
  • you did the (doubtful?) work at the wrong place instead of modifying propagateSTLAttributes() (and with ugly comments/printfs)

Which model and prompt did you use? What was the problem?

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.

3 participants