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.

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

Added windows support (#368) #369

Merged
merged 1 commit into from
Aug 5, 2024

Conversation

ermakov-oleg
Copy link
Contributor

@ermakov-oleg ermakov-oleg commented Aug 5, 2024

#368
I tried to build the CLI on a Windows GitHub runner and it seems like there are no problems.

I tested the built binary on one of the Windows machines:

PS C:\Users\...\Downloads> .\squawk-windows-x64.exe --version
squawk 1.1.2
PS C:\Users\...\Downloads> .\squawk-windows-x64.exe 'test.sql'
test.sql:1:0: warning: adding-field-with-default

   1 | ALTER TABLE foo ADD COLUMN bar TEXT NOT NULL DEFAULT "baz";

  note: Adding a field with a VOLATILE DEFAULT requires a table rewrite with an ACCESS EXCLUSIVE lock. In Postgres versions 11+, non-VOLATILE DEFAULTs can be added without a rewrite.
  help: Add the field as nullable, then set a default, backfill, and remove nullability.

find detailed examples and solutions for each rule at https://squawkhq.com/docs/rules
PS C:\Users\...\Downloads>

Copy link

netlify bot commented Aug 5, 2024

👷 Deploy request for squawkhq pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit b4ae5e8

Copy link
Collaborator

@chdsbd chdsbd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!!

@chdsbd chdsbd added the automerge automerge with kodiak label Aug 5, 2024
@sbdchd sbdchd added automerge automerge with kodiak and removed automerge automerge with kodiak labels Aug 5, 2024
@chdsbd chdsbd merged commit 3aedc07 into sbdchd:master Aug 5, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge automerge with kodiak
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants