Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade tooling #87

Merged
merged 9 commits into from
Nov 18, 2024
Merged

Upgrade tooling #87

merged 9 commits into from
Nov 18, 2024

Conversation

patinthehat
Copy link
Member

@patinthehat patinthehat commented Nov 18, 2024

upgrades the tooling, workflows, etc. to their latest versions.

Copy link

what-the-diff bot commented Nov 18, 2024

PR Summary

  • Updated Go Version in GitHub Workflow
    The Go programming language version used in our GitHub workflow has been updated from version 1.20 to 1.23. This will ensure that we are utilizing the most up-to-date features and security updates of the Go language.

  • Enhanced Pre-Commit Hook
    Our version control system now includes enhancements to its pre-commit hook. It can run tools named markdownlint and prettier on Markdown and JSON/YAML files, respectively, that are ready for commit. These tools will help enforce code styling and consistency rules.

  • Improved File Handling
    Two new functions have been added to the file helpers.go. These functions are FindExistingFile which helps to locate specific files, and GetCommandOutput, an improved way to gather output from executed commands.

  • Test for New File Handling Function
    To ensure the new FindExistingFile function operates correctly, a test file (helpers_test.go) has been created.

  • Increased Complexity Threshold for Linting
    The complexity score limit before the gocyclo linter issues a warning has been raised from 20 to 30 in the .golangci.yaml file. This allows for slightly more complex code before a warning is flagged.

  • Increased Required Go Version
    The Go version requirement has been raised to 1.22 in the go.mod file. This means that to build our code, Go version 1.22 or later is now required.

  • Updated Software Release Tool Configuration
    The configuration for Goreleaser, our tool for building and releasing new software versions, has been updated. The main entry path has been set to ./main.go.

  • Clean-Up and Organization of our Release Configuration
    Unnecessary lines and structures have been removed from the .goreleaser.yaml file. This will improve the clarity and readability of our configuration.

  • Improved Path Usage in Checksum Updates
    The method of updating checksums in the Taskfile.dist.yaml has been modified. This change ensures proper paths are used, leading to better accuracy and dependability of our checksums.

Copy link

codecov bot commented Nov 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 21.22%. Comparing base (4a069ee) to head (468892a).
Report is 39 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #87      +/-   ##
==========================================
- Coverage   26.09%   21.22%   -4.87%     
==========================================
  Files          18       41      +23     
  Lines        1556     2846    +1290     
==========================================
+ Hits          406      604     +198     
- Misses       1110     2201    +1091     
- Partials       40       41       +1     
Files with missing lines Coverage Δ
lib/support/helpers.go 13.64% <ø> (ø)

... and 40 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 58a278a...468892a. Read the comment docs.

@patinthehat patinthehat merged commit 4a36062 into main Nov 18, 2024
8 checks passed
@patinthehat patinthehat deleted the upgrade-tooling branch November 18, 2024 14:31
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.

1 participant