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

Automatically run code checks locally #374

Closed
shobhitagarwal1612 opened this issue Feb 18, 2020 · 6 comments
Closed

Automatically run code checks locally #374

shobhitagarwal1612 opened this issue Feb 18, 2020 · 6 comments
Labels
help wanted Extra attention is needed type: process Automated builds, test automation, etc.

Comments

@shobhitagarwal1612
Copy link
Member

From comment #370 (review)

  • Run code check on commit instead
  • Realtime warnings in Android Studio (probably by using a plugin)
@shobhitagarwal1612 shobhitagarwal1612 added type: process Automated builds, test automation, etc. priority: p2 labels Feb 18, 2020
@shobhitagarwal1612 shobhitagarwal1612 changed the title [Feature] Description Run code check on git commit Feb 18, 2020
@gino-m
Copy link
Collaborator

gino-m commented Mar 27, 2020

Hi @shobhitagarwal1612! Are we abandoning this one in favor of running checks manually, if I recall? If so we can close it.

@shobhitagarwal1612
Copy link
Member Author

Can we reduce the priority to p3 and mark it has help needed? Let's keep it open so we don't forget about it and re-visit it again sometime in future. Wdyt?

@gino-m gino-m added help wanted Extra attention is needed priority: p3 and removed priority: p2 labels Mar 29, 2020
@gino-m gino-m changed the title Run code check on git commit Automatically run code checks locally Mar 29, 2020
@gino-m
Copy link
Collaborator

gino-m commented Mar 29, 2020

Updated - also updated the description to be more broad, since we're not sure what the best solution is yet.

@gino-m gino-m added this to Ground Aug 30, 2022
@gino-m gino-m moved this to Icebox in Ground Aug 30, 2022
@anandwana001
Copy link
Collaborator

@shobhitagarwal1612
Do we need any pre-commit analysis?

@shobhitagarwal1612
Copy link
Member Author

I think it's a "good to have" feature but can leave it for now in favor of other high priority tasks.

@gino-m
Copy link
Collaborator

gino-m commented Dec 9, 2024

@anandwana001 We experimented with this awhile back and the latency introduced by the checks had a negative impact on developer velocity. I agree with @shobhitagarwal1612 we should punt on this.

@gino-m gino-m closed this as not planned Won't fix, can't repro, duplicate, stale Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed type: process Automated builds, test automation, etc.
Projects
No open projects
Status: Icebox
Development

No branches or pull requests

3 participants