Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move c2a-core crate source file to top dir for avoid misleading #30

Merged
merged 1 commit into from
Jul 26, 2023

Conversation

sksat
Copy link
Member

@sksat sksat commented Jul 26, 2023

概要

src/lib.rsc2a_core.rs に 移動・rename する

Issue

詳細

  • 現状,c2a-core をハック的に Rust のライブラリとして扱うためだけに src ディレクトリができていた
  • Rust の慣習に合わせるという意味ではこれもナシではないが,歪なディレクトリ構造を修正する #28 ですべてのソースファイルを src 以下に移すつもりがあるわけでもないし,誤解を産みがちだった
  • 今はハックを行っている数行の lib.rs があるのみであり,ディレクトリを分ける必然性があるわけではあまりない
  • 上記に加え,Cargo の機能でソースファイルの場所を明示的に指定できたため,ソースファイルをトップディレクトリに移す

検証結果

cargo build できればよし

影響範囲

c2a-core crate ライブラリの中身を記述する場所が変わる

@sksat sksat added enhancement New feature or request tools labels Jul 26, 2023
@sksat sksat requested a review from meltingrabbit July 26, 2023 07:00
@sksat sksat self-assigned this Jul 26, 2023
@sksat sksat merged commit 798f64b into develop Jul 26, 2023
24 checks passed
@sksat sksat deleted the feature/move-rs-to-top branch July 26, 2023 11:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request tools
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants