Skip to content
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

Remove parameters #808

Merged
merged 5 commits into from
Mar 1, 2023
Merged

Remove parameters #808

merged 5 commits into from
Mar 1, 2023

Conversation

mtgoncalves1
Copy link
Contributor

@mtgoncalves1 mtgoncalves1 commented Jan 24, 2023

Overview

Remove all the deprecated parameters.
PRs:
https://gitlab.com/architect-io/hub-api/-/merge_requests/691
https://gitlab.com/architect-io/architect-hub/-/merge_requests/690

Changes

Delete and update any code where parameters were used in the CLI, API and cloud.

Tests

Added parameters in the architect.yml of the hello-world example

parameters:
   test:
      default: test

$ architect dev
dev-command

$ architect-dev register
register-command

@mtgoncalves1 mtgoncalves1 self-assigned this Jan 24, 2023
@tjhiggins tjhiggins changed the title Remove parameters WIP: Remove parameters Jan 24, 2023
@tjhiggins
Copy link
Member

Adding WIP - not for the code, but we need the remaining customers to migrate first

@ryan-cahill ryan-cahill marked this pull request as draft January 24, 2023 17:15
Copy link
Member

@ryan-cahill ryan-cahill left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. I'll see where we stand on customer communication of this change tomorrow

@ryan-cahill ryan-cahill marked this pull request as ready for review February 15, 2023 15:12
@ryan-cahill ryan-cahill changed the title WIP: Remove parameters Remove parameters Feb 15, 2023
Merge remote-tracking branch 'origin/rc' into 404-remove-parameters
@tjhiggins tjhiggins merged commit 72cb793 into rc Mar 1, 2023
@tjhiggins tjhiggins deleted the 404-remove-parameters branch March 1, 2023 20:21
github-actions bot pushed a commit that referenced this pull request Mar 1, 2023
# [1.34.0-rc.2](v1.34.0-rc.1...v1.34.0-rc.2) (2023-03-01)

### Bug Fixes

* **spec:** Remove parameters ([#808](#808)) ([72cb793](72cb793))
github-actions bot pushed a commit that referenced this pull request Mar 7, 2023
# [1.34.0](v1.33.2...v1.34.0) (2023-03-07)

### Bug Fixes

* **cluster:create:** Make sure a cluster name is unique before creation ([#840](#840)) ([27d14f7](27d14f7))
* **deploy:** Add missing flags from register to deploy ([#849](#849)) ([61c3349](61c3349))
* **dev:restart:** Default --build to true ([#847](#847)) ([dbf8cc0](dbf8cc0))
* **dev:** Update description of environment flag to emphasize it is for local use only ([e9a66c4](e9a66c4))
* **init:** Depends on now supports dictionaries ([#848](#848)) ([30fedca](30fedca))
* **register:** Hard remove old cache directory before renaming ([#846](#846)) ([a9e2720](a9e2720))
* **spec:** Remove parameters ([#808](#808)) ([72cb793](72cb793))

### Features

* **dev:restart:** Add dev:restart command ([#842](#842)) ([93d10cf](93d10cf))
@github-actions
Copy link

github-actions bot commented Mar 7, 2023

🎉 This PR is included in version 1.34.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

3 participants