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

Add flutter-lints Dependency #9

Closed
Neha62-lit opened this issue Oct 3, 2021 · 4 comments · Fixed by #10 or #12
Closed

Add flutter-lints Dependency #9

Neha62-lit opened this issue Oct 3, 2021 · 4 comments · Fixed by #10 or #12
Assignees
Labels
Enhancement New feature or request

Comments

@Neha62-lit
Copy link
Contributor

Is your feature request related to a problem? Please describe.
Yes, In issue no #1 there is need of flutter-lint dependency, because we need to test our app using GitHub Actions.
Github Actions help us to automate our software development workflows in the same place we store code and collaborate on pull requests and issues. The speed of Github Actions is good, and its pricing is better than that of Travis.

Describe the solution you'd like
Linting is the process of analyzing a source code for any programming error, stylistic errors or any bug that can occur in the code. It is helpful, if you are working in a team, that way the whole team will follow some specific rules in the codebase. It's also helpful in identifying some common and uncommon mistakes that are made during coding like logical errors, unused variables etc.

Additional context
This is the package i propose to use here,
https://pub.dev/packages/flutter_lints

@Neha62-lit
Copy link
Contributor Author

Hey @OverPoweredDev, Please assign this issue to me because for completing issue #1, I have to complete this issue first.

@OverPoweredDev
Copy link
Owner

OverPoweredDev commented Oct 3, 2021

Sure, no problem! I've been meaning to get CI/CD stuff up for a long time, I just get super distracted with new features 😅

It'd be awesome if you could set linting and all up though, thanks! I'll put in the labels too so it counts

@OverPoweredDev
Copy link
Owner

I've merged the PR, good work on that! I'll close the issue once it's integrated with Github Actions

You can do it if you want, or leave it as is

@Neha62-lit
Copy link
Contributor Author

Okay

@OverPoweredDev OverPoweredDev linked a pull request Oct 4, 2021 that will close this issue
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

Successfully merging a pull request may close this issue.

2 participants