Skip to content

feat: Add Annotated Tag Push Support#1139

Open
fabiovincenzi wants to merge 3327 commits intofinos:mainfrom
fabiovincenzi:push-tags
Open

feat: Add Annotated Tag Push Support#1139
fabiovincenzi wants to merge 3327 commits intofinos:mainfrom
fabiovincenzi:push-tags

Conversation

@fabiovincenzi
Copy link
Contributor

Description:
This PR introduces end-to-end support for annotated tag pushes

  • Split the push pipeline into separate branchPushChain and tagPushChain.
  • Update getChain() to select the correct chain based on action.type and presence of action.tag.
  • Render a dedicated “Tag Details” table when viewing a tag push instead of the commit diff.

Fixes #986


Note: Restored from deleted fork

This PR recreates the original PR #1051, which was automatically closed due to accidental fork deletion.

** For discussions and reviews:** See the original PR #1051

All commits are identical to the original with preserved git history.

kriswest and others added 27 commits July 31, 2025 14:51
chore(deps): update github-actions - workflows - .github/workflows/ci.yml
…y-plugin-samples - plugins/git-proxy-plugin-samples/package.json
…mples-manager

chore(deps): update dependency @finos/git-proxy to ^1.19.2 - git-proxy-plugin-samples - plugins/git-proxy-plugin-samples/package.json
…ebase' into 946-associate-commits-by-email-rebase
fix(deps): update dependency axios to ^1.11.0 - git-proxy-cli - packages/git-proxy-cli/package.json
Co-authored-by: Kris West <[email protected]>
Signed-off-by: Andy Pols <[email protected]>
@netlify
Copy link

netlify bot commented Aug 4, 2025

Deploy Preview for endearing-brigadeiros-63f9d0 canceled.

Name Link
🔨 Latest commit 29f0646
🔍 Latest deploy log https://app.netlify.com/projects/endearing-brigadeiros-63f9d0/deploys/69aee84422227100081efcde

kriswest and others added 23 commits September 5, 2025 15:18
chore(deps): update github-actions - workflows - .github/workflows/dependency-review.yml
chore(deps): update dependency @types/node to ^22.18.1 - li-cli - experimental/li-cli/package.json
fix(deps): update npm - website - website/package.json
test: improve repo DB tests and CheckRepoInAuthList tests
…inos#1140)

* feat: create zod schema for config

* refactor: migrate config loader to Zod

* refactor: integrate Zod-based loader and pass config to services

* chore: ignore jsonschema in unused deps check

* fix: fix config schema

* feat: migrate configuration system from JSON Schema validation to QuickType

* fix: fix failing tests

* chore: remove unused deps

* fix: fix types checks errors

* feat: add banner on top of auto-generated types

* test: add tests for proxy

* test: increase tes coverage

* test: increase tes coverage

* test: increase test coverage for generated config

* test: increase test coverage for config

* test: increase test coverage

* test: increase test coverage

* refactor: remove unused variables

* chore: remove unused jsonschema dependency

* fix: instantiate Proxy class in tests instead of using class directly

* feat: add ci step for generate-config-types check

* chore: remove jsonschema

* style: fun format

* fix: add prettier formatting to generate-config-types script

* test: add test for default config

* chore: use deprecated annotation for ssl paths

* chore: re generate config types

---------

Co-authored-by: Jamie Slome <[email protected]>
Co-authored-by: Kris West <[email protected]>
Co-authored-by: Juan Escalada <[email protected]>
chore(deps): update dependency cypress to v15 - - package.json
fix: allowing empty diffs in scanDiff
@jescalada jescalada mentioned this pull request Feb 4, 2026
3 tasks
…n files

- Delete 94 stale .js/.jsx files that duplicated .ts/.tsx equivalents after JS->TS migration
- Apply tag push changes to post-processor/audit.ts (use RequestType.PULL enum)
- Apply tag push changes to PushRequests/components/PushesTable.tsx (tag-aware columns/search)
- Apply tag push changes to test/chain.test.ts (branchPushChain, tagPushChain tests)
- Apply tag push changes to test/testParsePush.test.ts (parseTag tests, tag exec tests)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for pushing tags git push --tags Add support for pushing tags git push --tags

7 participants