Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Generate sequence diagrams on request #448

Open
3 tasks done
EttyKitty opened this issue Dec 20, 2024 · 1 comment
Open
3 tasks done

Generate sequence diagrams on request #448

EttyKitty opened this issue Dec 20, 2024 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@EttyKitty
Copy link

EttyKitty commented Dec 20, 2024

Checklist

  • This is a new feature, not a refactoring suggestion
  • I have checked for this feature in the Sourcery documentation
  • I have checked there are no open requests referencing the same feature

Description

Right now, there is only a binary switch to control generation of sequence diagrams: you can either enable or disable them for all PR reviews.
I, personally, disabled them, as they take a lot of space and are not very useful for me to have in every PR.

What I want to suggest, is to add a "command" similar to $sourcery-ai review comment, that generates reviews on demand, something like $sourcery-ai diagram, that'll make the bot to respond and execute the default diagram generating behavior.
This way, you can disable diagram generation by default for all PRs and ask to generate them case by case, when you actually need them.

@EttyKitty EttyKitty added the enhancement New feature or request label Dec 20, 2024
@Hellebore
Copy link
Collaborator

Hi - thanks for adding this. We were just discussing adding a feature like this today - it is definitely on our roadmap.

@ruancomelli ruancomelli self-assigned this Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants