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

Flutter Analyze task name inconsistent with other task names #91

Open
chrisrogersFS opened this issue Jul 20, 2023 · 3 comments
Open

Flutter Analyze task name inconsistent with other task names #91

chrisrogersFS opened this issue Jul 20, 2023 · 3 comments
Labels
not-feasible Requested change is not feasible due to various reasons

Comments

@chrisrogersFS
Copy link

I apologize if this is a nitpick, but the naming used within Azure Devops for the analyze task requires the "Task" suffix to be used in a pipeline yaml, but none of the other tasks do.

FlutterBuild@0 vs FlutterAnalyzeTask@0

Not sure if this is intended or not, wanted to bring it to your attention.

@hey24sheep
Copy link
Owner

Hi, thank you for creating an issue and looking out.

I am aware of this and I wanted to change it but it would break a lot of builds which I am not comfortable with. I don't think it is worth breaking builds for no actual productive gain.

@hey24sheep hey24sheep added the not-feasible Requested change is not feasible due to various reasons label Jul 22, 2023
@P1Rh0
Copy link

P1Rh0 commented Nov 1, 2024

The above is fair enough, but can we make it more clear in the documentation that the naming is different for this?

@hey24sheep
Copy link
Owner

@P1Rh0 I will try to make a change in the readme, I will need a little bit of time, thanks

@hey24sheep hey24sheep reopened this Nov 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
not-feasible Requested change is not feasible due to various reasons
Projects
None yet
Development

No branches or pull requests

3 participants