search

magma / S1APTester

A test tool that simulates the s1aptest functionality of a LTE network
Other
53 stars 42 forks source link

chore: non-inclusive language #88

Closed lucasgonze closed 1 year ago

lucasgonze commented 1 year ago

Your Environment

Describe the Issue

The Snyk console hosted by Linux Foundation at https://security.lfx.linuxfoundation.org/#/a092M00001IkV6eQAF/non-inclusive-language has flagged usage of terms which may alienate developers.

To Reproduce

Search the repo for these terms:

Expected behavior

Screenshots

image

Additional context

The term "insane" is reported by Snyk but no longer exists in the code.

The term "whitehat" is only in a URL, and the URL points to Facebook, which is now an obsolete reference and needs to be fixed anyway.

lucasgonze commented 1 year ago

New developments:

  1. The S1AP code needs to reflect that I2C "slave" is official language from specification and just changing it is not a solution. We should wait for the the spec to change.
  2. We should have a policy to prevent new adoption of non-inclusive language. This can be based on https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=49decddd39e5f6132ccd7d9fdc3d7c470b0061bb or https://docs.zephyrproject.org/3.1.0/hardware/peripherals/i2c.html . It can be on the wiki.
lucasgonze commented 1 year ago

Policy: https://github.com/magma/magma/wiki/Inclusive-Language