CCExtractor / CCExtractor/ultimate_alarm_clock
Feature: [Discuss] the need for testing PR
- Lingua principale
- Dart
- Stelle
- 112
- Fork
- 227
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
## 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.
Guida per i contributori
Apri la guida per i contributori
Valutazione
Questa issue non è ancora stata valutata.