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

Remove Deploy Step #13

Merged
merged 1 commit into from
Aug 17, 2021
Merged

Remove Deploy Step #13

merged 1 commit into from
Aug 17, 2021

Conversation

TPRobots
Copy link
Contributor

@TPRobots TPRobots commented Aug 17, 2021

What

Removes the deploy step.

Why

The deploy step uses the GitHub Releases provider. We should create releases manually, and only use such release automation on private repos. Discussed with Emilio.


Pull request opened by github-pullrequestcreator.

Deploys use the GitHub Releases provider. These releases should be created manually. Discussed internally with Emilio.
@maxmanders maxmanders changed the title Feature/remove deploy step Remove Deploy Step Aug 17, 2021
@maxmanders maxmanders merged commit cecc0b9 into master Aug 17, 2021
@maxmanders maxmanders deleted the feature/remove-deploy-step branch August 17, 2021 10:34
dariogavranovic referenced this pull request in devrev/kafka-connect-dynamodb Mar 18, 2025
* Remove deploy step (#13)

Deploys use the GitHub Releases provider. These releases should be created manually. Discussed internally with Emilio.

Co-authored-by: Max Manders <[email protected]>

* Upgrade lo4j (#14)

* Table whitelist config parameter

integration tests

- updated kinesis adapter
- service endpoints config
- fixed credential config types

updated kinesis-adapter

use gson only

integration test task

test switch from init sync

table whitelist config parameter

kcl.table.billing.mode config parameter

add aws-java-sdk-sts dependency

trustpilot#5

params

cleaning

cleanup

docs

* fix for security vunrability move to log4j 2.15.0

Co-authored-by: Raimondas Tijūnaitis <[email protected]>
Co-authored-by: anl-trustpilot <[email protected]>
Co-authored-by: anl-trustpilot <[email protected]>

* Updates log4j to version 2.17.1 (trustpilot#15)

Co-authored-by: Emilio Larrambebere <[email protected]>

* Update_package_versions (trustpilot#21)

* update kcl and dynamo packages

* add dist property to travis file

* override abstract CW  methods

Co-authored-by: fal-trustpilot <[email protected]>

---------

Co-authored-by: Trustpilot Robot User <[email protected]>
Co-authored-by: Max Manders <[email protected]>
Co-authored-by: Raimondas Tijūnaitis <[email protected]>
Co-authored-by: anl-trustpilot <[email protected]>
Co-authored-by: anl-trustpilot <[email protected]>
Co-authored-by: Emilio Larrambebere <[email protected]>
Co-authored-by: fal-trustpilot <[email protected]>
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