oppia / oppia-android

A free, online & offline learning platform to make quality education accessible for all.
https://www.oppia.org
Apache License 2.0
309 stars 509 forks source link

Add CI check to ensure all TextViews have a style check #3972

Open BenHenning opened 2 years ago

BenHenning commented 2 years ago

Related issue: #3971. We need to make sure that TextViews have a centrally managed style set so that RTL/LTR settings can be properly managed. To do this, we need a CI check that runs a script which parses all layout files to ensure that TextView elements have a style set that corresponds to a local codebase style (i.e. uses @style/).

For reference, I suggest looking at the manifest transformation script since it parses & modifies XML today (so it's a good basis for the parsing needs of this new TextView script).

BenHenning commented 2 years ago

/cc @veena14cs

yash10019coder commented 2 years ago

@BenHenning I would like to work on this issue

BenHenning commented 2 years ago

@BenHenning I would like to work on this issue

Sounds good! Assigning this to you @yash10019coder.

Rd4dev commented 1 month ago

Unassigning myself to free it up for others. Will pick it up later if it's still available.