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

Update DDEV addon command to use ddev add-on get #53

Merged
merged 1 commit into from
Oct 27, 2024

Conversation

dermatz
Copy link
Collaborator

@dermatz dermatz commented Oct 27, 2024

Update commands and tests to use ddev add-on get instead of ddev get.

  • commands/host/frontend-update
    • Change ddev get to ddev add-on get for fetching the latest version of the addon.
  • install.yaml
    • Update pre-install actions to use ddev add-on get instead of ddev get.
  • tests/test.bats
    • Modify installation tests to use ddev add-on get instead of ddev get.
  • CHANGELOG.md
    • Add an entry to document the update to use ddev add-on get for versions of DDEV prior to v1.23.5.

Update commands and tests to use `ddev add-on get` instead of `ddev get`.

* **commands/host/frontend-update**
  - Change `ddev get` to `ddev add-on get` for fetching the latest version of the addon.
* **install.yaml**
  - Update pre-install actions to use `ddev add-on get` instead of `ddev get`.
* **tests/test.bats**
  - Modify installation tests to use `ddev add-on get` instead of `ddev get`.
* **CHANGELOG.md**
  - Add an entry to document the update to use `ddev add-on get` for versions of DDEV prior to v1.23.5.
Copy link

trunk-io bot commented Oct 27, 2024

⏱️ 5m total CI duration on this PR

Job Cumulative Duration Recent Runs
tests (HEAD) 4m 🟩
check-changelog 3s 🟩

settingsfeedbackdocs ⋅ learn more about trunk.io

@dermatz dermatz merged commit a6b4419 into main Oct 27, 2024
4 checks passed
@dermatz dermatz deleted the update-ddev-addon-command branch October 27, 2024 23:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant