CCExtractor / CCExtractor/ultimate_alarm_clock

Feature: [Discuss] the need for testing PR

Đang mở
#298 2 bình luận 0 reaction 1 người được giao Được @AdarshRawat1 nhận Xem trên GitHub
enhancement feat
Ngôn ngữ chính
Dart
Star
112
Fork
227
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

## Current Problems
- **Inconsistent code formatting, causing problem in detecting changes and testing PRs**
( Shows a lot of unnecessary changes in PR and may also introduce unwanted merge conflicts in future)
_Reference of such a event ( PR had to be reverted due to bugs) -_
Original PR - https://github.com/CCExtractor/ultimate_alarm_clock/pull/257
Revert - https://github.com/CCExtractor/ultimate_alarm_clock/pull/268

The problem is relevant for Beacon project as well [ https://github.com/CCExtractor/beacon ]
## Solution
- Adopt the official Dart formatting rules and use the `dartfmt` tool to automatically format Dart code on commit/push.
- Use of linting tools
- we can also use `flutter drive` for automating end-to-end UI testing and automated testing for unit tests and components as well

## impact
- `dartfmt` ensures consistent code style across the codebase for cleaner diffs.
- Testing ensures that bugs are detected before going into production

## Notes
- _Even if the testing might seem unnecessary at the moment for this application, but I strongly suggest using a linting or a standard formatting to ensure that changes made in PR are easier for maintainers to track and code base remains consistent._
- This issue will be divided it into subtasks.
- I would like to work on this.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.