Skip to content
This repository was archived by the owner on Dec 24, 2025. It is now read-only.

Conversation

@mocchira
Copy link
Contributor

@mocchira mocchira commented Aug 4, 2025

No description provided.

@mocchira mocchira self-assigned this Aug 4, 2025
@mocchira mocchira force-pushed the ysh/fix-ci-settings branch from 7fc89a9 to d072c45 Compare August 4, 2025 04:13
@mocchira mocchira changed the title chore: trigger CI chore: make github workflows work Aug 4, 2025
@mocchira mocchira marked this pull request as ready for review August 4, 2025 05:25
@mocchira mocchira requested a review from a team August 4, 2025 05:25
# Run tests with all features
- name: Run complete tests
run: cargo test --all-features --doc
run: cargo test --features "default,snapshot_builder,fs_import,url_import,web,os_exit" --doc
Copy link
Contributor Author

Choose a reason for hiding this comment

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

現状 --all-features は動作しないので、我々が利用している features のみ enabled にしてテストするよう変更

- name: web_features
run: cargo run --example web_features --features "web"
# - name: web_features
# run: cargo run --example web_features --features "web"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

breaking changes の影響で動作しないため、一旦コメントアウト

Copy link

Choose a reason for hiding this comment

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

理由をコメントとして残しておくと良さそう?ですかね?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ですね。
9022459
こちらで対応しました!

- name: websocket
run: cargo run --example websocket --features="web websocket"
# - name: websocket
# run: cargo run --example websocket --features="web websocket"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

同上

Copy link
Contributor Author

Choose a reason for hiding this comment

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

9022459 同上です

# Lint all the things
- name: Run clippy
run: cargo clippy --all-features
run: cargo clippy --features "default,snapshot_builder,fs_import,url_import,web,os_exit"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

現状 --all-features は動作しないので、我々が利用している features のみ enabled にしてテストするよう変更

@@ -0,0 +1,43 @@
name: Rust Tests
Copy link
Contributor Author

Choose a reason for hiding this comment

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

中身は tests.yml と同じです。
既存のワークフローを修正しただけでは、CI が動作しなかったのですが、新規ワークフロー追加する事で動作するようになったので、tests.yml を削除し、本ファイルを新規作成しています

Copy link

@k1LoW k1LoW left a comment

Choose a reason for hiding this comment

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

LGTM!ありがとうございます!

@mocchira mocchira merged commit a0538c1 into master Aug 5, 2025
5 checks passed
@mocchira mocchira deleted the ysh/fix-ci-settings branch August 5, 2025 01:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants