Skip to content

Conversation

@MarkoSagadin
Copy link
Collaborator

Description

As the title says.

Areas of interest for the reviewer

You can ignore everything in old-zephyr folder.

Checklist

  • My code follows the style guidelines as defined by IRNAS.
  • I have performed a self-review of my code.
  • My changes generate no new warnings.
  • I added/updated source code documentation for all newly added or changed functions.
  • I updated all customer-facing technical documentation.

After-review steps

  • I will merge PR by myself.

Copy link
Collaborator Author

MarkoSagadin commented Mar 17, 2025

@MarkoSagadin MarkoSagadin requested a review from NejcKle March 17, 2025 13:53
CHANGELOG.md Outdated
Comment on lines 11 to 12
- Migrate `zephyr` workflow to to `self-hosted-systemd` runners. `zephyr` workflow now uses Docker
containers to manage tooling and dependencies.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Migrate `zephyr` workflow to to `self-hosted-systemd` runners. `zephyr` workflow now uses Docker
containers to manage tooling and dependencies.
- Migrate `zephyr` workflow to `self-hosted-systemd` runners. `zephyr` workflow now uses Docker
containers to manage tooling and dependencies.

Base automatically changed from feature/pre-commit-workflow to main March 21, 2025 10:21
Zephyr workflows now use Docker containers to manager tooling and
environment.

The previous Zephyr workflow was deprecated and moved into `old-zephyr`
folder.
@MarkoSagadin MarkoSagadin force-pushed the feature/zephyr_with_docker branch from c8a9e28 to 08b1702 Compare March 21, 2025 10:22
@MarkoSagadin MarkoSagadin merged commit 6e12aad into main Mar 21, 2025
1 check passed
@MarkoSagadin MarkoSagadin deleted the feature/zephyr_with_docker branch March 21, 2025 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants