Skip to content

Addition of files used to run Woke word checker. - #322

Merged
lgirdwood merged 1 commit into
thesofproject:masterfrom
deb-intel:woke-addon
Feb 3, 2021
Merged

Addition of files used to run Woke word checker.#322
lgirdwood merged 1 commit into
thesofproject:masterfrom
deb-intel:woke-addon

Conversation

@deb-intel

Copy link
Copy Markdown
Collaborator

Signed-off-by: Deb Taylor deb.taylor@intel.com

Includes two files:
.wokeignore lists files to ignore when running Woke.
rules-woke.yaml is the customized file that must be run to accurately identify and correct non-inclusive terms.
This file is called by woke -c ./rules-woke.yaml

@lgirdwood

Copy link
Copy Markdown
Member

@xiulipan ready when you are.

@xiulipanxiulipan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I found some duplicated rules.

Comment threadrules-woke.yaml Outdated
@xiulipan

Copy link
Copy Markdown
Contributor

@deb-intel@lgirdwood I am fine with this configure files, please merge if you solve the duplicated rules above. Then I will work on this base for the woke.

@xiulipan

Copy link
Copy Markdown
Contributor

@deb-intel Good news, the public market action can work with extend rules. see xiulipan#1 (comment) for detail.
I will try to modify the wokeignore to avoid such issues. But it prove that our customized rules is working.

Comment thread.wokeignore Outdated
Signed-off-by: Deb Taylor <deb.taylor@intel.com>

@xiulipanxiulipan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.
PS: do we need this for other project?

@deb-intel

Copy link
Copy Markdown
CollaboratorAuthor

@xiulipan Are you referring to a particular project?

@lgirdwood

Copy link
Copy Markdown
Member

@xiulipan Are you referring to a particular project?

It's the source code for FW and tools etc, but we can do this after CNY when @xiulipan is back.

@lgirdwood
lgirdwood merged commit c6d6c58 into thesofproject:masterFeb 3, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@deb-intel@lgirdwood@xiulipan