Skip to content

fix(i18n-embed-fl): Fix return value for fl!() macro to always be String #185

fix(i18n-embed-fl): Fix return value for fl!() macro to always be String

fix(i18n-embed-fl): Fix return value for fl!() macro to always be String #185

Triggered via push February 1, 2024 08:28
Status Success
Total duration 57s
Artifacts

rust.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

27 warnings
Rustfmt
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2, actions-rs/toolchain@v1, actions-rs/cargo@v1. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Rustfmt
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions-rs/toolchain@v1, actions-rs/cargo@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Rustfmt
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Rustfmt
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Rustfmt
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Rustfmt
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Clippy
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions-rs/toolchain@v1, actions-rs/clippy@master. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Clippy
The following actions uses node12 which is deprecated and will be forced to run on node16: actions-rs/toolchain@v1, actions-rs/clippy@master. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Clippy
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Clippy
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Clippy
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Clippy
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Clippy: i18n-config/src/gettext.rs#L120
warning: this `impl` can be derived --> i18n-config/src/gettext.rs:120:1 | 120 | / impl Default for GettextAddLocation { 121 | | fn default() -> Self { 122 | | GettextAddLocation::Full 123 | | } 124 | | } | |_^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#derivable_impls = note: `#[warn(clippy::derivable_impls)]` on by default = help: remove the manual implementation... help: ...and instead derive it... | 104 + #[derive(Default)] 105 | pub enum GettextAddLocation { | help: ...and mark the default variant | 105 ~ #[default] 106 ~ Full, |
Clippy: i18n-config/src/gettext.rs#L120
warning: this `impl` can be derived --> i18n-config/src/gettext.rs:120:1 | 120 | / impl Default for GettextAddLocation { 121 | | fn default() -> Self { 122 | | GettextAddLocation::Full 123 | | } 124 | | } | |_^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#derivable_impls = note: `#[warn(clippy::derivable_impls)]` on by default = help: remove the manual implementation... help: ...and instead derive it... | 104 + #[derive(Default)] 105 | pub enum GettextAddLocation { | help: ...and mark the default variant | 105 ~ #[default] 106 ~ Full, |
Clippy: i18n-embed/src/gettext.rs#L89
warning: accessing first element with `language_ids.get(0)` --> i18n-embed/src/gettext.rs:89:28 | 89 | let language_id = *language_ids | ____________________________^ 90 | | .get(0) | |___________________^ help: try: `language_ids.first()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#get_first = note: `#[warn(clippy::get_first)]` on by default
Clippy: i18n-embed/src/lib.rs#L646
warning: accessing first element with `components.get(0)` --> i18n-embed/src/lib.rs:646:52 | 646 | let locale: Option<String> = match components.get(0) { | ^^^^^^^^^^^^^^^^^ help: try: `components.first()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#get_first
Clippy: src/main.rs#L82
warning: usage of an `Arc` that is not `Send` and `Sync` --> src/main.rs:82:55 | 82 | let cargo_i18n_localizer_rc: Arc<dyn Localizer> = Arc::new(cargo_i18n_localizer); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = note: `Arc<DefaultLocalizer<'_>>` is not `Send` and `Sync` as: = note: - the trait `Send` is not implemented for `DefaultLocalizer<'_>` = note: - the trait `Sync` is not implemented for `DefaultLocalizer<'_>` = help: consider using an `Rc` instead. `Arc` does not provide benefits for non `Send` and `Sync` types = note: if you intend to use `Arc` with `Send` and `Sync` traits = note: wrap the inner type with a `Mutex` or implement `Send` and `Sync` for `DefaultLocalizer<'_>` = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#arc_with_non_send_sync = note: `#[warn(clippy::arc_with_non_send_sync)]` on by default
Clippy: src/main.rs#L83
warning: usage of an `Arc` that is not `Send` and `Sync` --> src/main.rs:83:35 | 83 | let i18n_build_localizer_rc = Arc::new(i18n_build::localizer()) as Arc<dyn Localizer>; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = note: `Arc<DefaultLocalizer<'_>>` is not `Send` and `Sync` as: = note: - the trait `Send` is not implemented for `DefaultLocalizer<'_>` = note: - the trait `Sync` is not implemented for `DefaultLocalizer<'_>` = help: consider using an `Rc` instead. `Arc` does not provide benefits for non `Send` and `Sync` types = note: if you intend to use `Arc` with `Send` and `Sync` traits = note: wrap the inner type with a `Mutex` or implement `Send` and `Sync` for `DefaultLocalizer<'_>` = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#arc_with_non_send_sync
Clippy: src/main.rs#L141
warning: deref on an immutable reference --> src/main.rs:141:32 | 141 | .default_value(&*fallback_locale) | ^^^^^^^^^^^^^^^^^ help: if you would like to reborrow, try removing `&*`: `fallback_locale` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#borrow_deref_ref = note: `#[warn(clippy::borrow_deref_ref)]` on by default
Test Suite
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2, actions-rs/toolchain@v1, actions-rs/cargo@v1. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Test Suite
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions-rs/toolchain@v1, actions-rs/cargo@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Test Suite
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Test Suite
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Test Suite
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Test Suite
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Test Suite
/home/runner/work/cargo-i18n/cargo-i18n/i18n-build/Cargo.toml: `default-features` is ignored for tr, since `default-features` was not specified for `workspace.dependencies.tr`, this could become a hard error in the future
Test Suite
/home/runner/work/cargo-i18n/cargo-i18n/Cargo.toml: `default-features` is ignored for tr, since `default-features` was not specified for `workspace.dependencies.tr`, this could become a hard error in the future