-
Notifications
You must be signed in to change notification settings - Fork 15
Hookup hotshot #177
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
Closed
cptthura-alt
wants to merge
210
commits into
hookdeck:chore/use_pull_request_target
from
atlantis-marine:next
Closed
Hookup hotshot #177
cptthura-alt
wants to merge
210
commits into
hookdeck:chore/use_pull_request_target
from
atlantis-marine:next
Conversation
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
but not both
Bumps [github.com/gosimple/slug](https://github.com/gosimple/slug) from 1.9.0 to 1.14.0. - [Release notes](https://github.com/gosimple/slug/releases) - [Commits](gosimple/slug@v1.9.0...v1.14.0) --- updated-dependencies: - dependency-name: github.com/gosimple/slug dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]>
Bumps [github.com/spf13/cobra](https://github.com/spf13/cobra) from 1.0.0 to 1.8.1. - [Release notes](https://github.com/spf13/cobra/releases) - [Commits](spf13/cobra@v1.0.0...v1.8.1) --- updated-dependencies: - dependency-name: github.com/spf13/cobra dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]>
…es/github.com/spf13/cobra-1.8.1
…s been triggered by workflow_run
…ithub.com/spf13/cobra-1.8.1 chore(deps): bump github.com/spf13/cobra from 1.0.0 to 1.8.1
…dependabot/go_modules/github.com/gosimple/slug-1.14.0
…ithub.com/gosimple/slug-1.14.0 chore(deps): bump github.com/gosimple/slug from 1.9.0 to 1.14.0
chore: various dependency updates
Bumps [github.com/tidwall/pretty](https://github.com/tidwall/pretty) from 1.0.2 to 1.2.1. - [Commits](tidwall/pretty@v1.0.2...v1.2.1) --- updated-dependencies: - dependency-name: github.com/tidwall/pretty dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]>
Bumps [github.com/sirupsen/logrus](https://github.com/sirupsen/logrus) from 1.8.0 to 1.9.3. - [Release notes](https://github.com/sirupsen/logrus/releases) - [Changelog](https://github.com/sirupsen/logrus/blob/master/CHANGELOG.md) - [Commits](sirupsen/logrus@v1.8.0...v1.9.3) --- updated-dependencies: - dependency-name: github.com/sirupsen/logrus dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]>
…ithub.com/sirupsen/logrus-1.9.3 chore(deps): bump github.com/sirupsen/logrus from 1.8.0 to 1.9.3
Bumps [github.com/AlecAivazis/survey/v2](https://github.com/AlecAivazis/survey) from 2.2.9 to 2.3.7. - [Release notes](https://github.com/AlecAivazis/survey/releases) - [Commits](AlecAivazis/survey@v2.2.9...v2.3.7) --- updated-dependencies: - dependency-name: github.com/AlecAivazis/survey/v2 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]>
…ithub.com/tidwall/pretty-1.2.1 chore(deps): bump github.com/tidwall/pretty from 1.0.2 to 1.2.1
Bumps [github.com/gorilla/websocket](https://github.com/gorilla/websocket) from 1.4.2 to 1.5.3. - [Release notes](https://github.com/gorilla/websocket/releases) - [Commits](gorilla/websocket@v1.4.2...v1.5.3) --- updated-dependencies: - dependency-name: github.com/gorilla/websocket dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]>
Bumps [github.com/briandowns/spinner](https://github.com/briandowns/spinner) from 1.11.1 to 1.23.2. - [Release notes](https://github.com/briandowns/spinner/releases) - [Commits](briandowns/spinner@v1.11.1...v1.23.2) --- updated-dependencies: - dependency-name: github.com/briandowns/spinner dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]>
fix: Prevent lowercase when sluggify
…-logging Fix format string mismatch in OnConnectionWarning logging
…chore/beta-releases
chore: add beta release support across all package managers
…feat/cli-enrichment
…deck CLI Reference
add connection management to the CLI to support testing, debugging, getting started, and infra as code
Bumps [golang.org/x/sys](https://github.com/golang/sys) from 0.37.0 to 0.38.0. - [Commits](golang/sys@v0.37.0...v0.38.0) --- updated-dependencies: - dependency-name: golang.org/x/sys dependency-version: 0.38.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]>
…olang.org/x/sys-0.38.0 chore(deps): bump golang.org/x/sys from 0.37.0 to 0.38.0
- Remove connection_archive.go and connection_unarchive.go command files - Remove ArchiveConnection() and UnarchiveConnection() API client methods - Remove related tests from connections_test.go - Update documentation in README.md and REFERENCE.md to remove archive/unarchive references - Keep disable/enable commands as the primary lifecycle management commands - All 32 acceptance tests pass successfully Aligns with 2024-03-01 API changes where archive/unarchive were renamed to disable/enable
Remove deprecated archive/unarchive commands in favor of disable/enable
v1.2.0 release
…or Linux, Mac, and Windows
Bumps [golang.org/x/term](https://github.com/golang/term) from 0.36.0 to 0.37.0. - [Commits](golang/term@v0.36.0...v0.37.0) --- updated-dependencies: - dependency-name: golang.org/x/term dependency-version: 0.37.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]>
Add support for pinning Hookdeck project configuration to the current directory using the --local flag. This enables per-repository project configuration for better developer experience in multi-project workflows. Implementation: - Added --local flag to 'project use' command - Implemented UseProjectLocal() method to create/update local config - Added smart default: auto-updates local config when it exists - Validates --local and --config are mutually exclusive - Displays security warning when creating local config - Refactored config writing to reduce code duplication Testing: - Created comprehensive acceptance test suite - Added helper functions for temp directory management - Tests validate flag behavior, config creation, and security warnings - 2 tests passing in CI, 4 tests ready for CLI key testing Documentation: - Updated README with complete --local flag documentation - Explained configuration file precedence (--config > local > global) - Added security guidance for credential management - Included examples for all use cases Related: Previous PRs hookdeck#102, hookdeck#103 that removed --local flag
Add semi-automated testing workflow for project use tests that require human browser-based authentication. These tests ensure the --local flag works correctly with CLI key authentication. Implementation: - Created project_use_manual_test.go with 4 manual tests (build tag: manual) - Added RequireCLIAuthentication() helper that forces fresh login - Helper clears existing auth, runs 'hookdeck login', waits for user - Existing automated tests remain in project_use_test.go (run in CI) - Manual tests only run with: go test -tags=manual Testing workflow: - Tests clear authentication and require fresh CLI login - User completes browser authentication when prompted - Tests verify local config creation with project selection - Validates security warnings and smart default behavior Documentation: - Updated test/acceptance/README.md with manual test instructions - Explained difference between automated (CI) and manual tests - Provided examples of running manual tests locally - Documented which tests require human authentication Feature implementation: - Added security warning output in project_use.go for new local configs - Fixed config writing logic in config.go to properly handle new files - Refactored project_use_test.go to separate automated vs manual tests - Updated test comments to clarify CI vs local execution requirements Related: Completes semi-automated testing for --local flag feature
- Add type display in brackets [TYPE] for connection list view - Add 'Type:' field to connection get detailed view - Update tests to verify type fields are displayed correctly
… get - Add support for passing connection name to 'connection get' command - Implement resolveConnectionID() to accept both ID and name - Improve 404 error handling with user-friendly messages - Add tests for name-based lookup and error cases - Update command usage and help text to reflect name support
… of hookdeck-beta
…k-cli into feat/local-flag
- Added destinationURL and destinationCliPath to hasAnyDestinationFlag() check - Added destinationURL and destinationCliPath to hasDestinationConfigOnly check - Added destinationURL and destinationCliPath to hasDestinationConfigUpdate check - Added destinationCliPath handling in buildDestinationInputForUpdate() - Added comprehensive test suite for partial updates (TestConnectionUpsertPartialUpdates) This fixes the 422 error when running: hookdeck connection upsert <name> --destination-url <URL> Previously, the CLI wasn't detecting that destination config flags were provided, so it didn't fetch the existing connection to preserve source/destination references. The API requires both source and destination in every PUT request, so the CLI now automatically handles this by fetching the existing connection and merging changes.
fix: connection upsert with only destination config flags now works
Project Use --local Flag & Connection Management Improvements
…-generation Disable changelog generation in goreleaser configuration
…olang.org/x/term-0.37.0 chore(deps): bump golang.org/x/term from 0.36.0 to 0.37.0
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
brew install hook-up version