Skip to content

Enable GitHub Actions#106

Merged
renormalize merged 9 commits into
ai-dynamo:mainfrom
renormalize:main
Jul 11, 2025
Merged

Enable GitHub Actions#106
renormalize merged 9 commits into
ai-dynamo:mainfrom
renormalize:main

Conversation

@renormalize

Copy link
Copy Markdown
Contributor

We now use GitHub Actions for the following:

  • Build the operator and initc binaries.
  • Run all unit tests.
  • Generate CRDs and clients.
  • Generate API docs.
  • Run the formatter.
  • Run the linter.
  • Add license headers.

If the git tree is dirtied by any of the above steps, then the workflow fails. The PR author will have to commit changes in manually to ensure the workflow passes.


The following are still pending:

  • Workflow to build docker images for operator and initc.
  • Workflow to publish these docker images to a container registry.
  • Workflow to trigger and tag a release.

This PR also removes dead code, and cleans up the Makefiles.

renormalize and others added 9 commits July 10, 2025 23:51
Signed-off-by: Saketh Kalaga <51327242+renormalize@users.noreply.github.com>
* Removes targets repeated across each `Makefile`, where the target does
  not have anything tightly coupling it to its directory/module.

Signed-off-by: Saketh Kalaga <51327242+renormalize@users.noreply.github.com>
Signed-off-by: Saketh Kalaga <51327242+renormalize@users.noreply.github.com>
Signed-off-by: Saketh Kalaga <51327242+renormalize@users.noreply.github.com>
Signed-off-by: Saketh Kalaga <51327242+renormalize@users.noreply.github.com>
* Removed dead code.

* Fix other lint issues raised by `golangci-lint`.

Signed-off-by: Saketh Kalaga <51327242+renormalize@users.noreply.github.com>
Signed-off-by: Saketh Kalaga <51327242+renormalize@users.noreply.github.com>
Signed-off-by: Saketh Kalaga <51327242+renormalize@users.noreply.github.com>
Signed-off-by: Ron Kahn <rkahn@nvidia.com>
@renormalize renormalize merged commit 5ae566e into ai-dynamo:main Jul 11, 2025
1 check passed
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