Fix broken docker-compose link in quick start guide

[kavya-seth-vns]

Description

Fixes #3965

Update the docker-compose raw URL reference in the Quick Start guide.

The command in site/content/in-dev/unreleased/getting-started/quick-start.md
was updated to use a different raw GitHub URL path referencing the main branch.

curl -s https://raw.githubusercontent.com/apache/polaris/main/site/content/guides/quickstart/docker-compose.yml | docker compose -f - up

Both the old and the new URLs point to the same file. This change only updates the URL format used in the documentation.

This change only affects the unreleased documentation and does not modify documentation for older released versions such as 1.3.0.

[kavya-seth-vns]

Hi maintainers, I have fixed the issue with the quick start guide for version 1.3.0.
Please review the PR when you have time. Thank you!

[adutra]

@kavya-seth-vns there is a conflict apparently.

[dimas-b]

Thanks for your contribution, @kavya-seth-vns , but it looks like the same fix has already been merged in #3967 🤷

[dimas-b]

Both URLs point to the same file, but the new one is shorter, so +1 🙂

@MonkeyCanCode : WDYT?

[adnanhemani]

+1

[dimas-b]

Actually, on a second review pass, I get the old URL for the YAML file when I browse to the "raw" file via GH UI.

With that in mind, I believe the old URL is more reliable long-term and I'd prefer to keep it (I'll remove my approval for now).

Process:

• Open https://github.com/apache/polaris/blob/main/site/content/guides/quickstart/docker-compose.yml
• Click the "raw" button

[kavya-seth-vns]

Thanks for the clarification. The quickstart command already uses the original RAW URL for the docker-compose file, so no additional change was needed. Please let me know if you’d like me to update anything else.

[dimas-b]

@kavya-seth-vns : please update the PR description. I do not think it accurately reflects the actual change anymore.

[kavya-seth-vns]

@kavya-seth-vns : please update the PR description. I do not think it accurately reflects the actual change anymore.

Hi @dimas-b, I have updated the PR description to accurately reflect the current change. Please let me know if any further updates are needed. Thank you!

[dimas-b]

@kavya-seth-vns : The description still claims The documentation had incorrect command formatting [...] but the actual diff replaced one valid URL with another valid URL, as far as I can tell 🤔

as noted in #3968 (comment)

[kavya-seth-vns]

Hi @MonkeyCanCode,
All checks are passing and the requested changes have been addressed.
Could you please take a look when you have time? Thanks!

[MonkeyCanCode]

Hi @MonkeyCanCode,

All checks are passing and the requested changes have been addressed.

Could you please take a look when you have time? Thanks!

Hello,

I will check this later tonight.

Thanks,
Yong

[MonkeyCanCode]

Hi @MonkeyCanCode,
All checks are passing and the requested changes have been addressed.
Could you please take a look when you have time? Thanks!

Hello,

I will check this later tonight.

Thanks, Yong

@kavya-seth-vns thanks for the PR. As suggested by @dimas-b , we should update the description as that is not correct. This change will only update the url for unreleased and not older version (e.g. 1.3.0).

Thanks,
Yong

[kavya-seth-vns]

Hi @MonkeyCanCode,
All checks are passing and the requested changes have been addressed.
Could you please take a look when you have time? Thanks!

Hello,
I will check this later tonight.
Thanks, Yong

@kavya-seth-vns thanks for the PR. As suggested by @dimas-b , we should update the description as that is not correct. This change will only update the url for unreleased and not older version (e.g. 1.3.0).

Thanks, Yong

Thanks for the suggestion! I've updated the PR description and clarified that this change only affects the unreleased documentation.

[kavya-seth-vns]

Hi @dimas-b ,
All checks are passing and the requested changes have been addressed.
Could you please take a look when you have time? Thanks!

[dimas-b]

@kavya-seth-vns : The description still says Fix incorrect docker-compose raw URL in the Quick Start guide., but I do not think this is accurate. Both the old and the new URL in this PR are correct, IMHO. The only difference is in the minor URL path details in how they reference the main branch.

[dimas-b]

Also, I personally think the old (longer) URL is more correct than the proposed (new) shorter URL.

Cf. #3968 (comment)

[kavya-seth-vns]

Hi @dimas-b, I have updated the PR description to accurately reflect the current change. Please let me know if any further updates are needed. Thank you!

[jbonofre]

nit: I think it's better to keep an empty line after this line.