Skip to content

[certificates] Add CI hygiene checks, minor fixes on python hygiene checks#1435

Merged
mickmis merged 1 commit intointeruss:masterfrom
Orbitalize:fix_1352
Apr 16, 2026
Merged

[certificates] Add CI hygiene checks, minor fixes on python hygiene checks#1435
mickmis merged 1 commit intointeruss:masterfrom
Orbitalize:fix_1352

Conversation

@the-glu
Copy link
Copy Markdown
Contributor

@the-glu the-glu commented Apr 15, 2026

This PR add hygiene fixes for certificate manager.

A hygiene warning has been fixed (l -> logger)

It also side fix some minus elements on python checks:

  • Added --rm to docker run to remove docker containers
  • Fix PHONY tag for python-format

Comment thread deploy/operations/certificates-management/apply.py Dismissed
Comment thread deploy/operations/certificates-management/apply.py Dismissed
Copy link
Copy Markdown
Contributor

@mickmis mickmis left a comment

Choose a reason for hiding this comment

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

LGTM

@mickmis mickmis merged commit 1253906 into interuss:master Apr 16, 2026
12 checks passed
@mickmis mickmis deleted the fix_1352 branch April 16, 2026 06:24
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.

3 participants