CCExtractor / CCExtractor/ultimate_alarm_clock

Feature: [Discuss] the need for testing PR

Aperta
#298 2 commenti 0 reazioni 1 assegnatario Rivendicata da @AdarshRawat1 Vedi su GitHub
enhancement feat
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.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.