[PR #38] [MERGED] Modernize codebase #39

New issue

Closed

opened 2026-05-22 22:05:16 +01:00 by JakeStanger · 0 comments

JakeStanger commented

2026-05-22 22:05:16 +01:00

Owner

📋 Pull Request Information

Original PR: https://github.com/corn-config/corn/pull/38
Author: @puppymati
Created: 3/4/2025
Status: ✅ Merged
Merged: 3/5/2025
Merged by: @JakeStanger

Base: master ← Head: new-actions

📝 Commits (2)

f71fe09 Modernize ci workflows
cea8f00 Update dependencies and fix things

📊 Changes

12 files changed (+402 additions, -1285 deletions)

View changed files

📝 .github/workflows/build.yml (+31 -68)
📝 .github/workflows/release.yml (+11 -15)
📝 Cargo.lock (+326 -420)
📝 Cargo.toml (+1 -4)
➖ corn-cli/Cargo.lock (+0 -213)
📝 corn-cli/Cargo.toml (+6 -6)
➖ libcorn/Cargo.lock (+0 -540)
📝 libcorn/Cargo.toml (+22 -13)
📝 libcorn/src/de.rs (+1 -1)
📝 libcorn/src/lib.rs (+1 -1)
📝 libcorn/src/lua.rs (+2 -2)
📝 libcorn/src/parser.rs (+1 -2)

📄 Description

This PR modernizes the codebase with several key improvements:

Removed outdated Rust actions (rustup is now included in GitHub Actions)
Implemented nextest as test runner for significantly improved speed and industry best practices
Updated all actions to their latest versions
Fixed all clippy lints
Updated dependencies to latest versions
Resolved code compatibility issues with newer dependencies (notably, lua no longer requires lifetimes)
Fixed workspace lock file issues (previously duplicated and not properly utilized)

Note: While the main CI workflow changes are reliable, the release workflow has been updated but not fully tested. I considered removing these somewhat unreliable release workflows entirely but opted to update them for now.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/corn-config/corn/pull/38 **Author:** [@puppymati](https://github.com/puppymati) **Created:** 3/4/2025 **Status:** ✅ Merged **Merged:** 3/5/2025 **Merged by:** [@JakeStanger](https://github.com/JakeStanger) **Base:** `master` ← **Head:** `new-actions` --- ### 📝 Commits (2) - [`f71fe09`](https://github.com/corn-config/corn/commit/f71fe0905dba44aa7ff8078bcfadee77551232d3) Modernize ci workflows - [`cea8f00`](https://github.com/corn-config/corn/commit/cea8f00f522944dc305baf3d5c3cae86730c2366) Update dependencies and fix things ### 📊 Changes **12 files changed** (+402 additions, -1285 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/build.yml` (+31 -68) 📝 `.github/workflows/release.yml` (+11 -15) 📝 `Cargo.lock` (+326 -420) 📝 `Cargo.toml` (+1 -4) ➖ `corn-cli/Cargo.lock` (+0 -213) 📝 `corn-cli/Cargo.toml` (+6 -6) ➖ `libcorn/Cargo.lock` (+0 -540) 📝 `libcorn/Cargo.toml` (+22 -13) 📝 `libcorn/src/de.rs` (+1 -1) 📝 `libcorn/src/lib.rs` (+1 -1) 📝 `libcorn/src/lua.rs` (+2 -2) 📝 `libcorn/src/parser.rs` (+1 -2) </details> ### 📄 Description This PR modernizes the codebase with several key improvements: * Removed outdated Rust actions (rustup is now included in GitHub Actions) * Implemented nextest as test runner for significantly improved speed and industry best practices * Updated all actions to their latest versions * Fixed all clippy lints * Updated dependencies to latest versions * Resolved code compatibility issues with newer dependencies (notably, lua no longer requires lifetimes) * Fixed workspace lock file issues (previously duplicated and not properly utilized) Note: While the main CI workflow changes are reliable, the release workflow has been updated but not fully tested. I considered removing these somewhat unreliable release workflows entirely but opted to update them for now. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

JakeStanger

2026-05-22 22:05:16 +01:00