Skip to content
This repository was archived by the owner on Oct 9, 2025. It is now read-only.

Comments

fix: fixed invalid routes, removed docker compose, and some README up…#2

Merged
jburns24 merged 2 commits intomainfrom
fix-workflow-issues
Sep 7, 2025
Merged

fix: fixed invalid routes, removed docker compose, and some README up…#2
jburns24 merged 2 commits intomainfrom
fix-workflow-issues

Conversation

@jburns24
Copy link
Collaborator

@jburns24 jburns24 commented Sep 7, 2025

…dates

Copy link

@windsurf-bot windsurf-bot bot left a comment

Choose a reason for hiding this comment

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

💡 To request another review, post a new comment with "/windsurf-review".

Comment on lines +81 to 84
uses: goreleaser/goreleaser-action@v6
with:
distribution: goreleaser
version: latest
args: release --clean
Copy link

Choose a reason for hiding this comment

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

The 'version' parameter has been removed from the goreleaser-action configuration. This parameter is required even in v6 of the action to specify which goreleaser version to use.

Suggested change
uses: goreleaser/goreleaser-action@v6
with:
distribution: goreleaser
version: latest
args: release --clean
uses: goreleaser/goreleaser-action@v6
with:
distribution: goreleaser
version: latest
args: release --clean

Comment on lines +126 to +128
except KeyboardInterrupt:
print("\n🛑 Shutting down server gracefully...")
exit(0)
Copy link

Choose a reason for hiding this comment

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

The exit(0) call in the KeyboardInterrupt handler is redundant since Python would exit cleanly anyway when a KeyboardInterrupt is caught at the top level. Consider removing it for simplicity.

Suggested change
except KeyboardInterrupt:
print("\n🛑 Shutting down server gracefully...")
exit(0)
except KeyboardInterrupt:
print("\n🛑 Shutting down server gracefully...")

@jburns24 jburns24 merged commit fa52098 into main Sep 7, 2025
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant