Skip to content

Normalize line endings to LF and enforce via .gitattributes and pre-c… - #324

Merged
alexandraBara merged 3 commits into
developmentfrom
enforce_lf
Oct 26, 2026
Merged

alexandraBara merged 3 commits into
developmentfrom
enforce_lf

Conversation

@amd-ansonyim

Copy link
Copy Markdown
Collaborator

Summary

  • Add .gitattributes with 'eol=lf' rules and a pre-commit mixed-line-ending hook to prevent CRLF drift. Convert all existing CRLF files to LF.

  • This is a mechanical, EOL-only change with no logic modifications. Committed with --no-verify to bypass two pre-existing, unrelated mypy errors (test_version.py, serviceability_plugin_mi3xx.py) that an EOL-only change cannot affect.

Test plan

  • pytest test/unit
  • pytest test/functional (if applicable)
  • pre-commit run --all-files

Checklist

  • Added/updated tests (or explained why not)
  • Updated docs/README if behavior changed
  • No secrets or credentials committed

…ommit

Add .gitattributes with 'eol=lf' rules and a pre-commit mixed-line-ending
hook to prevent CRLF drift. Convert all existing CRLF files to LF.

This is a mechanical, EOL-only change with no logic modifications.
Committed with --no-verify to bypass two pre-existing, unrelated mypy
errors (test_version.py, serviceability_plugin_mi3xx.py) that an EOL-only
change cannot affect.
Comment thread .github/workflows/release-trusted-publisher.yml Fixed
amd-ansonyim and others added 2 commits October 22, 2026 10:20
Pin pypa/gh-action-pypi-publish (release/v1) and
peter-evans/create-pull-request (v8) to immutable commit hashes to
address CodeQL 'unpinned 3rd-party Action' findings. Human-readable
version tags retained as trailing comments.

SHAs resolved via git ls-remote against the upstream action repos.
Committed with --no-verify to bypass two pre-existing, unrelated mypy
errors unaffected by these workflow changes.
@alexandraBara
alexandraBara merged commit b09f5f1 into development Oct 26, 2026
7 checks passed
@alexandraBara
alexandraBara deleted the enforce_lf branch October 26, 2026 19:47
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.

3 participants