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 YAML and Markdown formatting #1153

Merged
merged 3 commits into from
Mar 26, 2024
Merged

Add YAML and Markdown formatting #1153

merged 3 commits into from
Mar 26, 2024

Conversation

le4ker
Copy link
Member

@le4ker le4ker commented Mar 19, 2024

Background

Adds prettier formatter to the repository in order to format YAML and Markdown files

Prettier will be installed after running make install and then the YAML and Markdown files can be formatted by running make fmt:

Testing

make install
make fmt

Needs this PAT update to run the tests successfully.

@le4ker le4ker requested review from a team March 19, 2024 12:27
@le4ker le4ker force-pushed the panos/chore/add-prettier branch from 563ac3c to a8d27d8 Compare March 19, 2024 12:31
@le4ker le4ker force-pushed the panos/chore/add-prettier branch 5 times, most recently from 7768734 to 2300c5c Compare March 20, 2024 11:32
@le4ker le4ker changed the title Add prettier formatter Add YAML and Markdown formatting Mar 20, 2024
Copy link
Contributor

@kjihso kjihso left a comment

Choose a reason for hiding this comment

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

This looks great, @le4ker !

Thanks for getting a hygiene improving solution in place.

@kjihso
Copy link
Contributor

kjihso commented Mar 25, 2024

@le4ker , @egibs

Does pat need to be updated to ignore package-lock.json? I'm getting the following failure with make test after cherry picking this pr's patch and running make fmt. I also see it in the test failure log here on the pr merge:

  Invalid Tests Summary
  	/home/runner/work/panther-analysis/panther-analysis/package-lock.json
  		Missing key: 'AnalysisType'
  
  	/home/runner/work/panther-analysis/panther-analysis/package.json
  		Missing key: 'AnalysisType'

https://github.com/panther-labs/panther-analysis/actions/runs/8422896110/job/23063255413

@kjihso
Copy link
Contributor

kjihso commented Mar 25, 2024

lol, should have looked at pat, first. It looks like this depends on: https://github.com/panther-labs/panther_analysis_tool/pull/470/files

@le4ker
Copy link
Member Author

le4ker commented Mar 26, 2024

lol, should have looked at pat, first. It looks like this depends on: https://github.com/panther-labs/panther_analysis_tool/pull/470/files

@kjihso correct, I have it mentioned in the PR description as well 😄

Needs panther-labs/panther_analysis_tool#470 PAT update to run the tests successfully.

@egibs egibs mentioned this pull request Mar 26, 2024
@le4ker le4ker merged commit ecfd1f7 into release Mar 26, 2024
5 checks passed
@le4ker le4ker deleted the panos/chore/add-prettier branch March 26, 2024 16:28
egibs pushed a commit that referenced this pull request Mar 26, 2024
* Remove API flags for CI Checks (#1159)

* Remove API flags for CI Checks

* Remove validate Workflow; move validation to upload Workflow

* chore: remove outdated license text (#1160)

* Bump PDH version and Pipfile.lock (#1161)

* custom enrichment LUT for TrailDiscover (#1162)

* Add detection for multiple emails for gitlab password reset (#1157)

* Add detection for multiple emails for gitlab password reset

* Update gitlab_audit_password_reset_multiple_emails.py

* Update rules/gitlab_rules/gitlab_audit_password_reset_multiple_emails.yml

* Update rules/gitlab_rules/gitlab_audit_password_reset_multiple_emails.yml

* Update rules/gitlab_rules/gitlab_production_password_reset_multiple_emails.yml

* Update rules/gitlab_rules/gitlab_production_password_reset_multiple_emails.yml

---------

Co-authored-by: Ariel Ropek <79653153+arielkr256@users.noreply.github.com>

* Update PAT to 0.44.0 (#1163)

* add prettier formatter (#1153)

---------

Co-authored-by: Lucy Suddenly <43256356+LucySuddenly@users.noreply.github.com>
Co-authored-by: Grant Joy <9968195+grantjoy@users.noreply.github.com>
Co-authored-by: Ariel Ropek <79653153+arielkr256@users.noreply.github.com>
Co-authored-by: Oleh Melenevskyi <767472+melenevskyi@users.noreply.github.com>
Co-authored-by: Panos Sakkos <panos.sakkos@panther.com>
Sign up for free to 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.

4 participants