feat: standardize build system and CI/CD automation - #133
Open
alexlovelltroy wants to merge 3 commits into
Open
Conversation
Signed-off-by: Alex Lovell-Troy <alovelltroy@lanl.gov>
- Added SPDX-FileCopyrightText and SPDX-License-Identifier headers to various files including workflows, scripts, and source code files to ensure compliance with licensing standards. - Updated documentation files and test files with appropriate licensing information. - Introduced a pre-commit configuration file to enforce code quality checks. Signed-off-by: Alex Lovell-Troy <alovelltroy@lanl.gov>
synackd
requested changes
Aug 31, 2026
synackd
changed the title
- Updated copyright year and format in all relevant files to "Copyright © 2026 OpenCHAMI a Series of LF Projects, LLC". - Modified the Makefile to include new variables for Go toolchain and GoReleaser versioning. - Added REUSE.toml for compliance with REUSE specifications. - Adjusted the installation and testing commands in the Makefile to utilize the new Go environment variable. - Removed outdated license files from JSON test cases and other directories. - Ensured consistent formatting and style across all Go source files and scripts. Signed-off-by: Alex Lovell-Troy <alovelltroy@lanl.gov>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This pull request introduces significant improvements to project automation, compliance, and developer workflow. It adds or updates CI/CD workflows for testing, linting, security, and releases, introduces a comprehensive
Makefilefor local automation, and brings the repository into SPDX license compliance with headers and documentation updates. Several configuration files and scripts are modernized for consistency with the CI environment, and developer documentation is enhanced to reflect these changes.CI/CD Workflow Improvements
lint.yaml), testing (test.yaml), and security (security.yaml), each with concurrency control and updated to use the latest actions and Go version fromgo.modfor consistent automation.Release.yml) to refine permissions, use the latest action versions, and align build steps with the newMakefileand Go toolchain management.Build and Development Automation
Makefileproviding targets for building, testing (including race and stress tests), linting, vulnerability scanning, and pre-PR checks, mirroring the CI workflows for local consistency.SPDX License Compliance and Documentation
LICENSEandLICENSES/MIT.txtfor clarity and compliance.README.mdto describe new automation and CI practices, and added aCHANGELOG.mdwith SPDX headers.Configuration and Linting
.golangci.ymland.pre-commit-config.yamlto enforce code style, linting, and pre-commit checks, and updated linting workflows to use these configurations.GoReleaser and Dependency Management
These changes collectively modernize the project's automation and compliance, making it easier for developers to contribute and for the project to maintain high quality and legal standards.
Checklist
make test(or equivalent) locally and all tests passgit commit -s) with my real name and email<filename>.licensesidecarLICENSES/directoryType of Change