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.

Sign up for GitHub

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

Fixes #1454 by setting the msrv #1463

Merged
merged 11 commits into from
Dec 28, 2023
Merged

Fixes #1454 by setting the msrv #1463

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
b003fef
sets MSRV for frb_codegen (wrongly to fail the tests) and implements …
patmuk Dec 21, 2023
8097db5
sets the correct MSRV
patmuk Dec 21, 2023
431a8a0
adds a (wrong) MSRV to frb_rust
patmuk Dec 21, 2023
5822266
changes the MSRV for frb_codegen to a wrong value, to test testing
patmuk Dec 21, 2023
f63eda9
changes testing for the MSRV by using an image with an explicit rust …
patmuk Dec 21, 2023
5584ca1
probably corrects the env var call
patmuk Dec 21, 2023
f1d760a
implemented env.FRB_MSRV_RUST_VERSION access differently
patmuk Dec 21, 2023
3e9918d
hardodes the MSRV
patmuk Dec 21, 2023
1b11265
sets the correct MSRV, but ci so that frb_rust passes, while frb_code…
patmuk Dec 21, 2023
72f8ad0
sets all correct values, so ci passes, as the final test
patmuk Dec 21, 2023
a880abe
small fixes
patmuk Dec 27, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 18 additions & 4 deletions .github/workflows/ci.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -104,6 +104,21 @@ jobs:
- run: ./frb_internal generate-website

# ----------------------------------- codegen -----------------------------------
msrv:
patmuk marked this conversation as resolved.
Show resolved Hide resolved
name: 'Test :: FRB Codegen :: MSRV'
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2

- name: Install Rust
uses: actions-rs/toolchain@v1
with:
toolchain: stable

- name: Run cargo msrv
run: |
cargo install cargo-msrv
cargo msrv --path frb_codegen verify

generate_run_frb_codegen_command_generate:
name: 'Generate :: FRB Codegen :: Command Generate'
Expand Down Expand Up @@ -146,7 +161,7 @@ jobs:
flutter-version: ${{ env.FRB_MAIN_FLUTTER_VERSION }}
architecture: x64
- uses: taiki-e/install-action@cargo-llvm-cov

# execute
- run: ./frb_internal generate-run-frb-codegen-command-generate --set-exit-if-changed --package ${{ matrix.package }} --coverage

Expand Down Expand Up @@ -339,7 +354,6 @@ jobs:
version: nightly
- image: ubuntu-latest
version: nightly-2023-12-16
# TODO(@fzyzcjy, @gutenfries): add MSRV here, as well as other locked versions as needed

steps:
# setup
Expand Down Expand Up @@ -367,8 +381,8 @@ jobs:
- uses: actions/upload-artifact@v4
with:
name: ${{ github.job }}--${{ matrix.info.image }}--${{ matrix.info.version }}--coverage
path: target/coverage

path: target/coverage
test_dart_native:
name: 'Test :: Dart :: Native'
runs-on: ${{ matrix.image }}
Expand Down
1 change: 1 addition & 0 deletions frb_codegen/Cargo.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ license.workspace = true
name = "flutter_rust_bridge_codegen"
repository.workspace = true
version.workspace = true
rust-version = "1.74.0"

[lib]
name = "lib_flutter_rust_bridge_codegen"
Expand Down
Loading