CCExtractor / CCExtractor/ultimate_alarm_clock

Feature: [Discuss] the need for testing PR

Open
#298 2 comments 0 reactions 1 assignee Claimed by @AdarshRawat1 View on GitHub
enhancement feat
Dominant language
Dart
Stars
112
Forks
227
PR merge metrics
No merged PRs in 30d

Description

## 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.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.