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

c2a-core hal crates: require c2a-core v4 #134

Merged
merged 1 commit into from
Dec 25, 2023

Conversation

sksat
Copy link
Member

@sksat sksat commented Oct 12, 2023

概要

#133 と同様に,hal の各 crate で要求する c2a-core(crate)のバージョン指定を major version のみにする

詳細

特定の c2a-core minor, patch version によらずに c2a-core hal の crate を使えるようになる

検証結果

CI が通ればよし

影響範囲

これにより,c2a-core hal crate で bindgen 経由で用いている C2A の関数群は major version で維持しなければならないインターフェースとなる(semver のバージョン判断をする API として扱うことになる)

補足

beta release は major version のみの指定では扱えないので,v4.0.0 のリリース以降

@sksat sksat added the enhancement New feature or request label Oct 12, 2023
@sksat sksat self-assigned this Oct 12, 2023
@sksat sksat mentioned this pull request Nov 2, 2023
8 tasks
@sksat sksat added this to the v4.2.0 milestone Nov 24, 2023
@meltingrabbit meltingrabbit changed the base branch from develop to main December 11, 2023 08:51
@sksat
Copy link
Member Author

sksat commented Dec 12, 2023

rebase

@sksat sksat force-pushed the feature/specify-major-version-in-hal-crates branch from 226d64e to 5e981f2 Compare December 12, 2023 08:03
@sksat sksat marked this pull request as ready for review December 12, 2023 08:06
@sksat
Copy link
Member Author

sksat commented Dec 25, 2023

rebase

@sksat sksat force-pushed the feature/specify-major-version-in-hal-crates branch from 5e981f2 to 67b94d5 Compare December 25, 2023 11:01
@sksat sksat merged commit c204e87 into main Dec 25, 2023
37 checks passed
@sksat sksat deleted the feature/specify-major-version-in-hal-crates branch December 25, 2023 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants