Skip to content

Remove standalone lintel-github-action build and release#98

Open
macalinao wants to merge 11 commits intomasterfrom
lintel-action-in-main
Open

Remove standalone lintel-github-action build and release#98
macalinao wants to merge 11 commits intomasterfrom
lintel-action-in-main

Conversation

@macalinao
Copy link
Contributor

Summary

  • Delete release-lintel-github-action.yml workflow
  • Remove ACTION_TAG trigger block from release-plz.yml
  • Remove lintel-github-action-static musl build from build.yml matrix
  • Skip lintel-github-action in release-plz (release = false, publish = false)
  • Remove lintel-github-action and lintel-github-action-static from nix packages and flake checks

Now that lintel-github-action is a library crate and lintel provides it as lintel github-action, it no longer needs its own build/release pipeline.

Test plan

  • nix flake check passes
  • nix build .#all succeeds without lintel-github-action
  • CI build matrix no longer includes lintel-github-action-static
  • release-plz no longer creates releases for lintel-github-action

…mand

Move the github-action functionality into the main lintel binary as a
hidden `lintel github-action` subcommand. The crate now exports a bpaf
struct (GithubActionArgs) and async runner instead of being a standalone
binary.

Key changes:
- Switch from lintel-validate to lintel-check (adds format checking)
- Add FormatDiagnostic::file_path() accessor in lintel-format
- Remove tokio dependency (runtime provided by caller)
- Update README to reflect new architecture
Now that lintel-github-action is a library subcommand (`lintel github-action`),
remove its separate build matrix entry, release workflow, release-plz trigger,
and nix packages. Skip its release and publish in release-plz.
…tibility

Move x86_64 and aarch64 GNU Linux builds from nix to cargo with
Debian Buster slim containers to ensure broad glibc compatibility.
Add smoke tests, glibc verification steps, and proper tar.gz packaging
for non-Windows targets.
Switch GNU Linux containers from debian:buster-slim to rust:bullseye
for Debian 11 glibc compatibility. Add RUSTFLAGS for symbol stripping
and single codegen unit in CI. Remove manual build dependency install
since rust:bullseye includes the toolchain.
- Move aarch64-apple-darwin and x86_64-apple-darwin from nix builds to
  cargo builds, avoiding non-portable nix store dylib references
- Add pull_request trigger for build workflow file changes so CI can
  be tested on branches before merging
- Nix builds now only handle musl static linking for Linux
Update version tags (vX.Y.Z and vX) on lintel-rs/action when
releasing lintel, so the GitHub Action tracks the latest release.
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