Skip to content

chore: add GitHub actions workflow#30

Open
freshtonic wants to merge 4 commits into
mainfrom
chore/github-actions
Open

chore: add GitHub actions workflow#30
freshtonic wants to merge 4 commits into
mainfrom
chore/github-actions

Conversation

@freshtonic
Copy link
Copy Markdown
Contributor

@freshtonic freshtonic commented Mar 20, 2026

What it says on the tin!

I assume the workflow will be picked up after this PR is merged.

@freshtonic freshtonic requested a review from auxesis March 20, 2026 11:08
@auxesis auxesis force-pushed the chore/github-actions branch from 22d6366 to 371711d Compare May 7, 2026 12:07
Copy link
Copy Markdown
Contributor

@auxesis auxesis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@freshtonic thanks for this.

I have fixed up GitHub Actions to run on this repo.

However, all of the checks now fail.

I have addressed the problems causing the checks to fail in #31. The fixes in #31 introduce massive churn into this codebase, so we should be very intentional about whether we do that or not.

Allow empty_line_after_outer_attr, large_enum_variant, and
unnecessary_unwrap to minimise source divergence from the upstream
sqlparser-rs fork.
@freshtonic freshtonic force-pushed the chore/github-actions branch from b34ca24 to a0bad6a Compare May 26, 2026 12:18
dtolnay/rust-toolchain@stable downloads have been intermittently
404ing from Azure-hosted runners. actions-rust-lang/setup-rust-toolchain
is actively maintained and includes built-in caching, so the separate
Swatinem/rust-cache step is no longer needed.
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.

2 participants