Skip to content
This repository has been archived by the owner on Oct 11, 2024. It is now read-only.

TT-11491: implement sync commands for API Template #146

Merged
merged 1 commit into from
May 31, 2024

Conversation

olamilekan000
Copy link

Description

change implements Tyk Sync commands (dump, publish, sync, update) for API Template

Related Issue

https://tyktech.atlassian.net/browse/TT-11491

Motivation and Context

Test Coverage For This Change

Screenshots (if appropriate)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • Make sure you are requesting to pull a topic/feature/bugfix branch (right side). If PRing from your fork, don't come from your master!
  • Make sure you are making a pull request against our master branch (left side). Also, it would be best if you started your change off our latest master.
  • Make sure you are updating CHANGELOG.md based on your changes.
  • My change requires a change to the documentation.
    • If you've changed APIs, describe what needs to be updated in the documentation.
  • I have updated the documentation accordingly.
  • If you've changed API models, please update CRDs.
    • make manifests
    • make helm
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • Check your code additions will not fail linting checks:
    • gofmt -s -w .
    • go vet ./...
    • golangci-lint run

@olamilekan000 olamilekan000 requested a review from a team as a code owner May 23, 2024 02:58
@olamilekan000 olamilekan000 requested review from buraksekili and removed request for a team May 23, 2024 02:58
cli-publisher/dashboard.go Outdated Show resolved Hide resolved
cli-publisher/dashboard.go Outdated Show resolved Hide resolved
cli-publisher/dashboard.go Outdated Show resolved Hide resolved
cli-publisher/dashboard.go Outdated Show resolved Hide resolved
cli-publisher/dashboard.go Show resolved Hide resolved
clients/objects/assets.go Outdated Show resolved Hide resolved
clients/objects/assets.go Show resolved Hide resolved
clients/objects/assets.go Outdated Show resolved Hide resolved
cmd/dump.go Outdated Show resolved Hide resolved
tyk-vcs/getter.go Outdated Show resolved Hide resolved
@olamilekan000 olamilekan000 force-pushed the TT-11491-sync-tempate-with-commands branch from f0c80cd to 687d524 Compare May 24, 2024 12:25
@olamilekan000 olamilekan000 force-pushed the TT-11491-sync-tempate-with-commands branch from 687d524 to af6b5f7 Compare May 30, 2024 12:15
@olamilekan000 olamilekan000 force-pushed the TT-11491-sync-tempate-with-commands branch from af6b5f7 to cd78b2a Compare May 30, 2024 13:11
Copy link

Quality Gate Passed Quality Gate passed

Issues
5 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@singhpr singhpr merged commit e9a19b7 into master May 31, 2024
19 checks passed
@singhpr singhpr deleted the TT-11491-sync-tempate-with-commands branch May 31, 2024 08:18
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.

3 participants