fix: command framework polish, CI fixes, and CR_PAT auth#23
Merged
Conversation
- CI: 4 jobs (lint/build/test+docker) with race detection, coverage artifact, and ghcr.io push on main - Release: on v* tag — vet+test, Docker push with semver+latest tags, auto-generated release notes via softprops/action-gh-release - Dockerfile: proxy ARGs for local builds behind filtered internet - docker-compose.yml: proxy args passthrough from .env - Makefile: proxy-forwarding docker-build target - .env.example: commented proxy vars Closes #9
covdata tool conflicts with -race on Go 1.22 runner. Replace with -cover flag for inline coverage output.
go.mod requires go 1.25.3 but Dockerfile used 1.22-alpine, causing go mod download to fail with version mismatch.
covdata tool conflicts with -race on Go 1.22 runner for packages without test files, causing CI to fail with exit code 1. Coverage tracking moved to issue #21.
…, dupe prevention - PermissionMember now checks Store.GetTeamsForUser (non-members rejected at registry level) - handleSubmit/handleStatus support --team flag for multi-team users - New /standup cancel command aborts active conversation - New /standup list command shows user's teams - activeOrNewSession prevents duplicate daily submissions - Help output updated with cancel/list entries - 4 new tests: CancelNoConversation, CancelActive, ListTeams, SubmitDuplicateSession Closes #22
Default GITHUB_TOKEN lacks permission to create org packages. Switching to a Personal Access Token (CR_PAT) with write:packages scope.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
All commits on feat/ci-pipeline that weren't in the initial merge:
CI fixes
Command framework polish (#22)
--team "Team Name"with submit/status/standup cancelcommand to abort active conversation/standup listcommand to show user's teamsFiles
Closes #22