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.

Sign up for GitHub

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

Jump to bottom

DX 관련 - 린트 설정 제안 드립니다. #2

Closed
fridayCCC opened this issue Dec 29, 2023 · 2 comments · Fixed by #12
Closed

DX 관련 - 린트 설정 제안 드립니다. #2

fridayCCC opened this issue Dec 29, 2023 · 2 comments · Fixed by #12
Assignees
@brownglasses
Labels
🔧세팅 프로젝트 세팅 관련 작업입니다.

Comments

@fridayCCC
Copy link
Contributor

요약

  • flutter_lints, import_sorter 설정으로 린트 설정을 통일했으면합니다.

이유

  • 린트 설정을 간략화 할 수 있고, import_sorter은 웹에서는 기본 린트설정에서 plugin으로 작성할 수 있는데, flutter에서는 보이지 않아 제안해봅니다.
  • 앱이 커질 때 파일에서 서로서로 import 해오는데, 이 때 순서를 통일해두면 파악이 더 쉽더라구요.

참고

https://deku.posstree.com/ko/flutter/linter/#flutter_lints
@fridayCCC fridayCCC added the 🤔의논 의논이 필요할 것 같아요 label Dec 29, 2023
@fridayCCC fridayCCC added this to the local login boilerplate milestone Dec 29, 2023
@brownglasses brownglasses self-assigned this Jan 1, 2024
@brownglasses brownglasses added 🔧세팅 프로젝트 세팅 관련 작업입니다. 🧵하는중 현재 작업 중입니다. and removed 🤔의논 의논이 필요할 것 같아요 labels Jan 1, 2024
@brownglasses
Copy link
Contributor

brownglasses commented Jan 1, 2024 •
edited
Loading

https://pub.dev/packages/import_sorter

image

image

@brownglasses
Copy link
Contributor

flutter 자주 사용하는 lint:

linter:
rules:
- prefer_const_constructors
- always_declare_return_types
- always_require_non_null_named_parameters
- avoid_types_on_closure_parameters
- avoid_annotating_with_dynamic
- avoid_escaping_inner_quotes
- avoid_function_literals_in_foreach_calls
- avoid_private_typedef_functions
- combinators_ordering
- curly_braces_in_flow_control_structures
- lowercase_with_underscores
- omit_local_variable_types
- annotate_overrides
- sort_constructors_first
- unawaited_futures
- directives_ordering
- unnecessary_final
- unnecessary_parenthesis
- conditional_uri_does_not_exist
- prefer_single_quotes
- await_only_futures
- always_use_package_imports
- avoid_empty_else
- comment_references
- prefer_void_to_null
- use_key_in_widget_constructors

brownglasses added a commit that referenced this issue Jan 2, 2024
@brownglasses
feat: 린트 추가
5315533 
일단 자주 사용하는 lint 를 추가해보았습니다. 후에 필요하면 또 추가하면 될 것 같습니다

resolve: #2
brownglasses added a commit that referenced this issue Jan 2, 2024
@brownglasses
feat: impoert_srter 추가
5096016 
dev_dependdencids 에 import_sorter 을 추가하였습니다.
터미널에 flutter pub run import_sorter:main 를 치면 import sorter 가 실행됩니다

resolve : #2
@brownglasses brownglasses linked a pull request Jan 2, 2024 that will close this issue
2 dx 관련 린트 설정 제안 드립니다 #12
Merged
@brownglasses brownglasses mentioned this issue Jan 2, 2024
Merged
@fridayCCC fridayCCC removed the 🧵하는중 현재 작업 중입니다. label Jan 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@brownglasses brownglasses

Labels
🔧세팅 프로젝트 세팅 관련 작업입니다.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

2 dx 관련 린트 설정 제안 드립니다 TaskWeaver/frontend_app
2 participants
@brownglasses @fridayCCC