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

Add .editorconfig and detection for Google's Container-Optimized OS #1392

Merged
merged 2 commits into from May 14, 2024

Conversation

ianchesal
Copy link
Contributor

@ianchesal ianchesal commented May 10, 2023

The .editorconfig file works for most modern editors. It ensures the indentation standards for the repository are consistent with what's in CONTRIBUTING.md. See here for more settings.

Updated include/osdetection so it recognizes Google's Container-Optimized OS.

Running this on a COS instance in Google Cloud I get:

[+] Initializing program
------------------------------------
  - Detecting OS...                                           [ DONE ]
  - Checking profiles...                                      [ DONE ]

  ---------------------------------------------------
  Program version:           3.0.8
  Operating system:          Linux
  Operating system name:     Container-Optimized OS from Google
  Operating system version:  101
  Kernel version:            5.15.107+
  Hardware platform:         x86_64
  Hostname:                  ian-cos-cis-compliance-testing

Lovely!

Modern editors read this file and use it to enforce formatting
consistency in files for a project.

See: https://editorconfig.org/

Settings were made accoring to what is written in the CONTRIBUTING.md
file.
@mboelen mboelen merged commit 68c9976 into CISOfy:master May 14, 2024
@mboelen
Copy link
Member

mboelen commented May 14, 2024

Thanks, both changes merged!

@mboelen mboelen self-assigned this May 14, 2024
@mboelen
Copy link
Member

mboelen commented May 14, 2024

Good one about the .editorconfig, did not know it yet. 🚀

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

2 participants