Skip to content

Commit

Permalink
actually fix silencing squawk stdin (#211)
Browse files Browse the repository at this point in the history
The previous fix was incorrect and broke stdin
  • Loading branch information
cdignam-segment authored May 11, 2022
1 parent afffabf commit a86531b
Show file tree
Hide file tree
Showing 5 changed files with 11 additions and 5 deletions.
6 changes: 6 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

## [Unreleased]

## v0.11.3 - 2022-05-11

## Fixed

- incorrectly silenced stdin

## v0.11.2 - 2022-05-11

## Fixed
Expand Down
2 changes: 1 addition & 1 deletion Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion cli/Cargo.toml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
[package]
name = "squawk"
version = "0.11.1"
version = "0.11.3"
authors = ["Steve Dignam <[email protected]>"]
edition = "2018"
license = "GPL-3.0"
Expand Down
4 changes: 2 additions & 2 deletions cli/src/reporter.rs
Original file line number Diff line number Diff line change
Expand Up @@ -147,10 +147,10 @@ pub fn check_files(
let sql = get_sql_from_stdin()?;
// ignore stdin if it's empty.
if sql.trim().is_empty() {
info!("ignoring empty stdin");
} else {
let path = stdin_path.unwrap_or_else(|| "stdin".into());
process_violations(&sql, &path)?;
} else {
info!("ignoring empty stdin");
}
}

Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "squawk-cli",
"version": "0.11.1",
"version": "0.11.3",
"description": "linter for PostgreSQL, focused on migrations",
"repository": "[email protected]:sbdchd/squawk.git",
"author": "Steve Dignam <[email protected]>",
Expand Down

0 comments on commit a86531b

Please sign in to comment.