Skip to content

Commit

Permalink
Merge branch 'master' into filter_generator_by_min_max_creatures
Browse files Browse the repository at this point in the history
# Conflicts:
#	.github/workflows/sonarcloud.yml
#	Cargo.toml
#	src/db/db_communicator.rs
  • Loading branch information
RakuJa committed Nov 16, 2023
2 parents b9a04d6 + 7785ed4 commit 7b4d6f9
Show file tree
Hide file tree
Showing 5 changed files with 33 additions and 40 deletions.
14 changes: 14 additions & 0 deletions .github/workflows/clippy.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
on: push
name: Clippy check

# Make sure CI fails on all warnings, including Clippy lints
env:
RUSTFLAGS: "-Dwarnings"

jobs:
clippy_check:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- name: Run Clippy
run: cargo clippy --all-targets --all-features
18 changes: 18 additions & 0 deletions .github/workflows/fmt.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
on: pull_request

name: Rustfmt

jobs:
format:
runs-on: ubuntu-latest
permissions:
pull-requests: write
steps:
- uses: actions/checkout@v4
- uses: dtolnay/rust-toolchain@nightly
with:
components: rustfmt
- uses: mbrobbel/rustfmt-check@master
with:
token: ${{ secrets.GITHUB_TOKEN }}
mode: review
38 changes: 0 additions & 38 deletions .github/workflows/sonarcloud.yml
Original file line number Diff line number Diff line change
@@ -1,38 +0,0 @@
name: SonarCloud analysis

on:
push:
branches: [ "master" ]
pull_request:
branches: [ "master" ]
workflow_dispatch:

permissions:
pull-requests: read # allows SonarCloud to decorate PRs with analysis results

jobs:
Analysis:
runs-on: ubuntu-latest

steps:
- name: Analyze with SonarCloud

# You can pin the exact commit or the version.
# uses: SonarSource/sonarcloud-github-action@de2e56b42aa84d0b1c5b622644ac17e505c9a049
uses: sonarsource/sonarcloud-github-action@master
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # Needed to get PR information
SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }} # Generate a token on Sonarcloud.io, add it to the secrets of this repo with the name SONAR_TOKEN (Settings > Secrets > Actions > add new repository secret)
with:
# Additional arguments for the sonarcloud scanner
args:
# Unique keys of your project and organization. You can find them in SonarCloud > Information (bottom-left menu)
# mandatory
-Dsonar.projectKey=RakuJa_BYBE
-Dsonar.organization=rakuja
#-Dsonar.projectBaseDir=app
# Comma-separated paths to directories containing main source files.
#-Dsonar.sources= # optional, default is project base directory
#-Dsonar.tests= # optional. For more info about Code Coverage, please refer to https://docs.sonarcloud.io/enriching/test-coverage/overview/
# Adds more detail to both client and server-side analysis logs, activating DEBUG mode for the scanner, and adding client-side environment variables and system properties to the server-side log of analysis report processing.
#-Dsonar.verbose= # optional, default is false
1 change: 0 additions & 1 deletion Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ validator = {version="0.16.1", features = ["derive"]}
utoipa = { version = "4", features = ["actix_extras"] }
utoipa-swagger-ui = { version = "4", features = ["actix-web"] }

redis = {version = "0.23.0-beta.1", features = ["json"]}
sqlx = { version = "0.7.2", features = ["runtime-async-std", "sqlite"] }
mini-moka = "0.10.2"

Expand Down
2 changes: 1 addition & 1 deletion src/db/db_communicator.rs
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ async fn from_raw_to_creature(conn: &Pool<Sqlite>, raw: &RawCreature) -> Creatur
}

pub async fn fetch_creatures(conn: &Pool<Sqlite>) -> Result<Vec<Creature>, Error> {
let creatures = sqlx::query_as::<_, RawCreature>("SELECT * FROM CREATURE_TABLE")
let creatures = sqlx::query_as::<_, RawCreature>("SELECT * FROM CREATURE_TABLE ORDER BY name")
.fetch_all(conn)
.await;
match creatures {
Expand Down

0 comments on commit 7b4d6f9

Please sign in to comment.