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

Mob/chipsec refactor2 #2091

Merged
merged 4 commits into from
Mar 22, 2024
Merged

Mob/chipsec refactor2 #2091

merged 4 commits into from
Mar 22, 2024

Conversation

npmitche
Copy link
Member

Refactoring of the chipset.py file to help readability/maintainability.

chipsec/library/register.py Fixed Show fixed Hide fixed
chipsec/modules/tools/vmm/hv/synth_kbd.py Fixed Show fixed Hide fixed
@@ -34,7 +34,7 @@
from chipsec.file import *
from chipsec.module_common import *
from chipsec.hal.vmm import VMM
from chipsec.defines import *
from chipsec.library.defines import *

Check notice

Code scanning / CodeQL

'import *' may pollute namespace Note

Import pollutes the enclosing namespace, as the imported module
chipsec.library.defines
does not define '__all__'.
chipsec/chipset.py Fixed Show fixed Hide fixed
chipsec/modules/tools/vmm/ept_finder.py Show resolved Hide resolved
chipsec/library/device.py Fixed Show fixed Hide fixed
chipsec/chipset.py Outdated Show resolved Hide resolved
docs/sphinx/development/Sample-Module-Code.rst Outdated Show resolved Hide resolved
chipsec/library/result_deltas.py Fixed Show fixed Hide fixed
chipsec/library/result_deltas.py Fixed Show fixed Hide fixed
chipsec/library/result_deltas.py Fixed Show fixed Hide fixed
chipsec/library/result_deltas.py Fixed Show fixed Hide fixed
chipsec/library/result_deltas.py Fixed Show fixed Hide fixed
tests/hardware/test_generic.py Fixed Show fixed Hide fixed
tests/software/util.py Fixed Show fixed Hide fixed
chipsec/library/result_deltas.py Fixed Show fixed Hide fixed
chipsec/library/register.py Fixed Show fixed Hide fixed
chipsec/modules/tools/vmm/hv/synth_dev.py Fixed Show fixed Hide fixed
@npmitche npmitche force-pushed the mob/chipsec_refactor2 branch 2 times, most recently from 5d949da to a5a306a Compare March 21, 2024 20:26
@Sae86 Sae86 merged commit 0e8192c into chipsec:main Mar 22, 2024
42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants