Skip to content

give functionality to use codeflash:skip in label like other bots #289

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Saga4
Copy link
Contributor

@Saga4 Saga4 commented Jun 5, 2025

No description provided.

@Saga4 Saga4 added the codeflash:skip Skip CF run for the particular PR label Jun 5, 2025
@Saga4 Saga4 requested a review from misrasaurabh1 June 5, 2025 18:07
@misrasaurabh1
Copy link
Contributor

@Saga4 how are users expected to use this?

@Saga4
Copy link
Contributor Author

Saga4 commented Jun 5, 2025

@Saga4 how are users expected to use this?

As roboflow asked to have optimization suggestions on their long running PR only once it gets stabilized is a genuine ask.
We can achieve this either by having some keyword in commit msg or PR title, or can do it through labels to pause the codeflash untill the PR is stabilizes. Let me know your thoughts?

@misrasaurabh1
Copy link
Contributor

i am fine with it but i don't this is the best UI. People would have to think and add labels which does not come naturally. So users will only add this once they get spammed and irritated, which means it is already too late.
After this users will forget to remove this which will disable codeflash permanently for that PR.
Is there a better way to do this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
codeflash:skip Skip CF run for the particular PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants