Skip to content

ci(workflows): create test workflow#43

Merged
argl merged 10 commits intomainfrom
test-publish-npm
Nov 12, 2025
Merged

ci(workflows): create test workflow#43
argl merged 10 commits intomainfrom
test-publish-npm

Conversation

@caugner
Copy link
Contributor

@caugner caugner commented Oct 30, 2025 •
edited
Loading

Description

Adds a test workflow that:

  1. Runs cargo fmt and cargo clippy.
  2. Simulates the publish-npm workflow.

Motivation

Ensure that we would catch issues before the actual release.

Additional details

See diff @ 832146c: https://www.diffchecker.com/Q8HqHGRi/

Related issues and pull requests

Follow-up of:

Blocked by:

@caugner caugner force-pushed the test-publish-npm branch 3 times, most recently from 3e68424 to 009d33d Compare October 30, 2025 11:34
@caugner caugner changed the title test(publish-npm): run on pull-request fix(publish-npm): install npm with sudo Oct 30, 2025
@caugner caugner marked this pull request as ready for review October 30, 2025 11:36
@caugner caugner requested a review from a team as a code owner October 30, 2025 11:36
@caugner caugner requested a review from argl October 30, 2025 11:36
@caugner caugner marked this pull request as draft October 30, 2025 15:26
@caugner caugner changed the title fix(publish-npm): install npm with sudo ci(workflows): add test workflow Oct 30, 2025
@caugner caugner changed the title ci(workflows): add test workflow ci(workflows): create test workflow Oct 31, 2025
caugner
caugner commented Oct 31, 2025
View reviewed changes
@caugner caugner marked this pull request as ready for review November 12, 2025 14:01
@argl argl merged commit cb910c5 into main Nov 12, 2025
4 checks passed
@argl argl deleted the test-publish-npm branch November 12, 2025 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@argl argl argl approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@caugner @argl