8000 Remove deprecated version field by tiborrr · Pull Request #4459 · distribution/distribution · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Remove deprecated version field #4459

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

Merged
merged 4 commits into from
Sep 13, 2024
Merged

Conversation

tiborrr
Copy link
Contributor
@tiborrr tiborrr commented Sep 6, 2024

Copy link
Member
@milosgajdos milosgajdos left a comment

Choose a reason for hiding this comment

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

@thaJeztah
Copy link
Member

Looks like the example is still written for compose v1 (docker-compose), which is EOL, so probably should be updated to use docker compose (space instead of hyphen).

@milosgajdos
Copy link
Member

@tiborrr mind updating

```consonle
$ docker-compose up -d
```

@thaJeztah
Copy link
Member

Looks like there's a couple of other places as well;

git grep 'docker-compose[^\.-]'
docs/content/about/deploying.md:$ docker-compose up -d
docs/content/recipes/apache.md:$ docker-compose up -d
docs/content/recipes/nginx.md:$ docker-compose up -d
releases/v3.0.0-alpha.1.toml:* Use docker-compose spec v3 in nginx receipt ([#3872](https://github.com/distribution/distribution/pull/3872))
releases/v3.0.0-alpha.1.toml:  * [`c624b9ed`](https://github.com/distribution/distribution/commit/c624b9eda7d193856216ee0013468777a16f2e8a) Use docker-compose spec v3 in nginx receipt

@tiborrr
Copy link
Contributor Author
tiborrr commented Sep 7, 2024

@thaJeztah @milosgajdos

deploying.md updated
apache.md updated
nginx.md updated
v3.0.0-alpha.1.toml does not require updating as this shows changes that have been in the past and I do not think this requires updating.

Signed-off-by: Tibor <tiborcasteleijn@gmail.com>
Signed-off-by: Tibor <tiborcasteleijn@gmail.com>
Signed-off-by: Tibor <tiborcasteleijn@gmail.com>
@tiborrr
Copy link
Contributor Author
tiborrr commented Sep 13, 2024

@thaJeztah can you approve this pull request with the changes I have made?

Copy link
Member
@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

oh! thought I did

LGTM, thanks!

@milosgajdos milosgajdos merged commit 2314320 into distribution:main Sep 13, 2024
16 checks passed
@tiborrr tiborrr deleted the patch-1 branch September 13, 2024 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0