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.

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

feat: Automated Dart Analyzer Verification for Pull Request #671

Closed
4 tasks done
j-j-gajjar opened this issue Jan 30, 2023 · 0 comments · Fixed by #761
Closed
4 tasks done

feat: Automated Dart Analyzer Verification for Pull Request #671

j-j-gajjar opened this issue Jan 30, 2023 · 0 comments · Fixed by #761
Labels
Feature request Requesting a new feature that's not implemented yet

Comments

@j-j-gajjar
Copy link
Contributor

Type

Functionality

Issue

Currently, there is no automated process in place to verify the Dart analyzer whenever a reviewer merges a pull request. This increases the risk of merging code that contains syntax or semantic errors, leading to potential bugs and decreased code quality.

Feature

A feature request is submitted to implement an automated process that runs the Dart analyzer on the code in a pull request before it is merged. This would help ensure that the code is free of errors and maintain the high quality of the codebase.

Motivation

Implementing this feature would improve the quality and reliability of the codebase by catching errors early in the development process. It would also save time for reviewers who would otherwise need to manually run the Dart analyzer on each pull request.

Additional context

Related to #665

Acknowledgements

  • I have searched the existing issues and this is a new and no duplicate or related to another open issue.
  • I have written a short but informative title.
  • I filled out all of the requested information in this issue properly.
  • The issue is related solely to the ReVanced Manager
@j-j-gajjar j-j-gajjar added the Feature request Requesting a new feature that's not implemented yet label Jan 30, 2023
@j-j-gajjar j-j-gajjar changed the title feat: Automated Dart Analyzer Verification for Pull Request Merges Feature Request feat: Automated Dart Analyzer Verification for Pull Request Jan 30, 2023
validcube added a commit to validcube/revanced-manager that referenced this issue Feb 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature request Requesting a new feature that's not implemented yet
Projects
None yet
1 participant