Merge branch 'main' into feat/security

SergioRibera · web-flow · commit 67ba5dcc25a7 · 2024-07-09T20:07:45.000-04:00
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -1,111 +1,37 @@
-name: CI
+name: "Continuous integration"
 on:
   push:
-    branches:
-      - main
   pull_request:
-    branches:
-      - main
+
 jobs:
-  build:
-    name: Build
+  ci:
+    name: CI checks
     runs-on: ubuntu-latest
     strategy:
       matrix:
         rust:
           - stable
           - beta
-          - nightly
-          - 1.45.0
     steps:
-      - name: Check out code
-        uses: actions/checkout@v2
-      - name: Install Rust toolchain
-        uses: actions-rs/toolchain@v1
-        with:
-          toolchain: ${{ matrix.rust }}
-          profile: minimal
-          components: clippy
-          override: true
-      - name: Cache Rust dependencies
-        uses: Swatinem/rust-cache@v1
-      - name: Install cargo-make
-        uses: actions-rs/cargo@v1
-        with:
-          command: install
-          args: cargo-make
-      - name: Build
-        if: ${{ matrix.rust != 'stable' }}
-        uses: actions-rs/cargo@v1
-        with:
-          command: make
-          args: ci-build
-      - name: Build and lint
+      - uses: actions/checkout@v4
+      - uses: actions-rust-lang/setup-rust-toolchain@v1
         if: ${{ matrix.rust == 'stable' }}
-        uses: actions-rs/cargo@v1
-        with:
-          command: make
-          args: ci-lint
-  format:
-    name: Format
-    runs-on: ubuntu-latest
-    steps:
-      - name: Check out code
-        uses: actions/checkout@v2
-      - name: Install Rust toolchain
-        uses: actions-rs/toolchain@v1
-        with:
-          toolchain: stable
-          profile: minimal
-          components: rustfmt
-          override: true
-      - name: Cache Rust dependencies
-        uses: Swatinem/rust-cache@v1
-      - name: Install cargo-make
-        uses: actions-rs/cargo@v1
-        with:
-          command: install
-          args: cargo-make
-      - name: Check format
-        uses: actions-rs/cargo@v1
         with:
-          command: make
-          args: ci-format
-  test:
-    name: Test
-    runs-on: ubuntu-latest
-    strategy:
-      matrix:
-        rust:
-          - stable
-          - beta
-          - nightly
-          - 1.45.0
-    steps:
-      - name: Check out code
-        uses: actions/checkout@v2
-      - name: Install Rust toolchain
-        uses: actions-rs/toolchain@v1
+          toolchain: "1.61.0" # TODO: Pull this from Cargo.toml?
+      - uses: actions-rust-lang/setup-rust-toolchain@v1
         with:
+          components: clippy, rustfmt
           toolchain: ${{ matrix.rust }}
-          profile: minimal
-          components: ""
-          override: true
-      - name: Cache Rust dependencies
-        uses: Swatinem/rust-cache@v1
-      - name: Install cargo-make
-        uses: actions-rs/cargo@v1
-        with:
-          command: install
-          args: cargo-make
-      - name: Run tests and generate coverage report
-        uses: actions-rs/cargo@v1
+      - uses: taiki-e/install-action@v2
         with:
-          command: make
-          args: ci-test
-      - name: Upload to codecov.io
+          tool: cargo-msrv,cargo-tarpaulin
+      - run: ./scripts/ci/format
+      - run: ./scripts/ci/check
+      - run: ./scripts/ci/test
+      - run: cargo msrv verify
+        if: ${{ matrix.rust == 'stable' }}
+      - uses: coverallsapp/github-action@v2
         if: ${{ matrix.rust == 'stable' }}
-        uses: codecov/codecov-action@v1
         with:
-          fail_ci_if_error: true
-
+          file: lcov.info
+          format: lcov
diff --git a/.gitignore b/.gitignore
@@ -2,4 +2,9 @@ target
 Cargo.lock
 
 # Backups from `cargo fmt`
-*.rs.bk
+*.rs.bk
+
+# Coverage report
+lcov.*
+*.lcov
+*.profraw
diff --git a/Cargo.toml b/Cargo.toml
@@ -1,15 +1,16 @@
 [package]
 name = "preferences"
-version = "3.0.0"
+version = "2.0.1-dev.0"
 authors = ["Andy Barron <AndrewLBarron@gmail.com>"]
 edition = "2021"
-rust-version = "1.61.0"
+rust-version = "1.61.0" # syn
 
 description = "Read and write user-specific application data (in stable Rust)"
 documentation = "https://docs.rs/preferences"
 repository = "https://github.com/AndyBarron/preferences-rs"
 readme = "README.md"
 keywords = ["preferences", "user", "data", "persistent", "storage"]
+categories = ["config"]
 license = "MIT-0"
 
 [features]
diff --git a/LICENSE.txt b/LICENSE.txt
@@ -1,21 +1,16 @@
-The MIT License (MIT)
+MIT No Attribution
 
-Copyright (c) 2016 Andy Barron
+Copyright 2024 Andy Barron
 
-Permission is hereby granted, free of charge, to any person obtaining a copy
-of this software and associated documentation files (the "Software"), to deal
-in the Software without restriction, including without limitation the rights
-to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-copies of the Software, and to permit persons to whom the Software is
-furnished to do so, subject to the following conditions:
+Permission is hereby granted, free of charge, to any person obtaining a copy of this
+software and associated documentation files (the "Software"), to deal in the Software
+without restriction, including without limitation the rights to use, copy, modify,
+merge, publish, distribute, sublicense, and/or sell copies of the Software, and to
+permit persons to whom the Software is furnished to do so.
 
-The above copyright notice and this permission notice shall be included in all
-copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
-SOFTWARE.
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED,
+INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A
+PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
+HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
+SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
diff --git a/Makefile.toml b/Makefile.toml
diff --git a/README.md b/README.md
@@ -1,14 +1,19 @@
 # preferences
 _Read and write user-specific application data in Rust_
 
-[![crates.io: preferences](https://img.shields.io/crates/v/preferences.svg?label=crates.io%3A%20preferences)](https://crates.io/crates/preferences)
-[![Linux & OS X build status](https://img.shields.io/travis/AndyBarron/preferences-rs.svg?label=Linux%20%26%20OS%20X%20builds)](https://travis-ci.org/AndyBarron/preferences-rs)
-[![Windows build status](https://img.shields.io/appveyor/ci/AndyBarron/preferences-rs.svg?label=Windows%20builds)](https://ci.appveyor.com/project/AndyBarron/preferences-rs)
+[![Crates.io Version](https://img.shields.io/crates/v/preferences)](https://crates.io/crates/preferences)
+![Crates.io MSRV](https://img.shields.io/crates/msrv/preferences)
+[![Crates.io License](https://img.shields.io/crates/l/preferences)](https://github.com/andybarron/preferences-rs/blob/main/LICENSE.txt)
+![Crates.io Total Downloads](https://img.shields.io/crates/d/preferences)
+
+[![docs.rs](https://img.shields.io/docsrs/preferences)](https://docs.rs/preferences)
+[![GitHub Actions Workflow Status](https://img.shields.io/github/actions/workflow/status/andybarron/preferences-rs/ci.yaml)](https://github.com/andybarron/preferences-rs/actions/workflows/ci.yaml)
+[![Coveralls](https://img.shields.io/coverallsCoverage/github/andybarron/preferences-rs)](https://coveralls.io/github/andybarron/preferences-rs)
 
 ## Documentation
 https://docs.rs/preferences
 
 ## Installation
-Add the following to your `Cargo.toml`:
-
-`preferences = "^1.1.0"`
+```sh
+cargo add preferences
+```
diff --git a/scripts/ci/check b/scripts/ci/check
@@ -0,0 +1,2 @@
+#!/usr/bin/env sh
+cargo clippy --all-targets -- -Dwarnings
diff --git a/scripts/ci/format b/scripts/ci/format
@@ -0,0 +1,2 @@
+#!/usr/bin/env sh
+cargo fmt -- --check --verbose
diff --git a/scripts/ci/test b/scripts/ci/test
@@ -0,0 +1,2 @@
+#!/usr/bin/env sh
+cargo tarpaulin -v --all-features --ignore-tests --out Lcov
diff --git a/scripts/release b/scripts/release
@@ -0,0 +1,2 @@
+#!/usr/bin/env sh
+cargo release "$@"
diff --git a/src/lib.rs b/src/lib.rs
@@ -1,3 +1,5 @@
+#![warn(clippy::all, clippy::pedantic, clippy::nursery, clippy::cargo)]
+#![allow(clippy::enum_glob_use)]
 //! *Read and write user-specific application data*
 //!
 //! This crate allows Rust developers to store and retrieve user-local preferences and other
@@ -189,8 +191,8 @@ use std::string::FromUtf8Error;
 pub mod security;
 
 const DATA_TYPE: AppDataType = AppDataType::UserConfig;
-static PREFS_FILE_EXTENSION: &'static str = ".prefs.json";
-static DEFAULT_PREFS_FILENAME: &'static str = "prefs.json";
+static PREFS_FILE_EXTENSION: &str = ".prefs.json";
+static DEFAULT_PREFS_FILENAME: &str = "prefs.json";
 
 /// Generic key-value store for user data.
 ///
@@ -241,7 +243,7 @@ impl std::error::Error for PreferencesError {
 
 impl From<serde_json::Error> for PreferencesError {
     fn from(e: serde_json::Error) -> Self {
-        PreferencesError::Json(e)
+        Self::Json(e)
     }
 }
 
@@ -250,19 +252,19 @@ impl From<FromUtf8Error> for PreferencesError {
         let kind = ErrorKind::InvalidData;
         let msg = "Preferences file contained invalid UTF-8";
         let err = io::Error::new(kind, msg);
-        PreferencesError::Io(err)
+        Self::Io(err)
     }
 }
 
 impl From<std::io::Error> for PreferencesError {
     fn from(e: std::io::Error) -> Self {
-        PreferencesError::Io(e)
+        Self::Io(e)
     }
 }
 
 impl From<AppDirsError> for PreferencesError {
     fn from(e: AppDirsError) -> Self {
-        PreferencesError::Directory(e)
+        Self::Directory(e)
     }
 }
 
@@ -295,21 +297,27 @@ pub trait Preferences: Sized {
     /// Saves the current state of this object. Implementation is platform-dependent, but the data
     /// will be local to the active user.
     ///
-    /// # Failures
+    /// # Errors
     /// If a serialization or file I/O error (e.g. permission denied) occurs.
     fn save<S: AsRef<str>>(&self, app: &AppInfo, key: S) -> Result<(), PreferencesError>;
     /// Loads this object's state from previously saved user data with the same `key`. This is
     /// an instance method which completely overwrites the object's state with the serialized
     /// data. Thus, it is recommended that you call this method immediately after instantiating
     /// the preferences object.
     ///
-    /// # Failures
+    /// # Errors
     /// If a deserialization or file I/O error (e.g. permission denied) occurs, or if no user data
     /// exists at that `path`.
     fn load<S: AsRef<str>>(app: &AppInfo, key: S) -> Result<Self, PreferencesError>;
     /// Same as `save`, but writes the serialized preferences to an arbitrary writer.
+    ///
+    /// # Errors
+    /// If a write or serialization error occurs.
     fn save_to<W: Write>(&self, writer: &mut W) -> Result<(), PreferencesError>;
     /// Same as `load`, but reads the serialized preferences from an arbitrary writer.
+    ///
+    /// # Errors
+    /// If a read or deserialization error occurs.
     fn load_from<R: Read>(reader: &mut R) -> Result<Self, PreferencesError>;
 }
 
@@ -359,18 +367,19 @@ where
 /// This makes no guarantees that the specified directory path actually *exists* (though you can
 /// easily use `std::fs::create_dir_all(..)`). Returns `None` if the directory cannot be determined
 /// or is not available on the current platform.
+#[must_use]
 pub fn prefs_base_dir() -> Option<PathBuf> {
     get_data_root(AppDataType::UserConfig).ok()
 }
 
 #[cfg(test)]
 mod tests {
-    use {AppInfo, Preferences, PreferencesMap};
+    use super::{AppInfo, Preferences, PreferencesMap};
     const APP_INFO: AppInfo = AppInfo {
         name: "preferences",
         author: "Rust language community",
     };
-    const TEST_PREFIX: &'static str = "tests/module";
+    const TEST_PREFIX: &str = "tests/module";
     fn gen_test_name(name: &str) -> String {
         TEST_PREFIX.to_owned() + "/" + name
     }

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+#!/usr/bin/env sh`
	`2`	`+cargo clippy --all-targets -- -Dwarnings`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+#!/usr/bin/env sh`
	`2`	`+cargo fmt -- --check --verbose`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+#!/usr/bin/env sh`
	`2`	`+cargo tarpaulin -v --all-features --ignore-tests --out Lcov`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+#!/usr/bin/env sh`
	`2`	`+cargo release "$@"`