docs icon indicating copy to clipboard operation
docs copied to clipboard

Clean cherry-pick: MariaDB guide update for Debian 12

Open DHBR2 opened this issue 4 months ago • 6 comments

This PR manually cherry-picks and commits the corrected MariaDB guide for Debian 12 Renamed guide folder and image from Debian 9 to Debian 12 Removed outdated image Resolved cherry-pick conflicts and committed cleanly This PR supersedes #7309 and #7320, which encountered cherry-pick and commit history issues. Ready for review and merge.

DHBR2 avatar Aug 12 '25 17:08 DHBR2

Hi @DHBR2 the Markdown links check is failing because there is a reference to Debian 9 guide. https://www.linode.com/docs/guides/list-tables-in-mysql-and-mariadb/ This link is in the "Before you Begin section" for this test to pass.

Rajakavitha1 avatar Aug 14 '25 11:08 Rajakavitha1

Tested and validated the MariaDB guide on Debian 12 mariadb12.txt

Rajakavitha1 avatar Aug 14 '25 12:08 Rajakavitha1

So, are you saying that I have to update a different document that is not part of the assigned changes in order to make this PR successful? Let me introduce myself - I work for Andy Stevens as a Technical Writer and am trying to learn the process for getting PRs to pass all checks. The Issue:

  • The PR is failing a "Markdown links check"
  • The failure is due to a broken or outdated link to a Debian 9 guide (https://www.linode.com/docs/guides/list-tables-in-mysql-and-mariadb/)
  • This link exists in the "Before you Begin" section of a document I didn’t modify
  • Even though this broken link isn't part of my changes, it's causing my PR to fail

I’m happy to fix it and add it to the PR, but I wanted to understand if there is an established process for handling this type of situation? My original assignment was to update the one document’s mention of Debian 9 and make relevant changes to the subject matter - no other documentation updates were mentioned.

Is fixing unrelated broken links typically the contributor’s responsibility, or is there another way to handle this? From: Rajakavitha Kodhandapani @.> Sent: Thursday, August 14, 2025 4:32 AM To: linode/docs @.> Cc: Hoober, Dianna @.>; Mention @.> Subject: Re: [linode/docs] Clean cherry-pick: MariaDB guide update for Debian 12 (PR #7321)

[https://avatars.githubusercontent.com/u/37059749?s=20&v=4]Rajakavitha1 left a comment (linode/docs#7321)https://urldefense.com/v3/__https:/github.com/linode/docs/pull/7321*issuecomment-3188122276__;Iw!!GjvTz_vk!UyH7xlcmZeQ5QjV0rEK7Ku5dv-hU7q9RkYkjuQKdaXNAteT8a_goXcW3rCu6RIdgqmP78I_3xI-pvleuxF_Q5zk$

Hi @DHBR2https://urldefense.com/v3/__https:/github.com/DHBR2__;!!GjvTz_vk!UyH7xlcmZeQ5QjV0rEK7Ku5dv-hU7q9RkYkjuQKdaXNAteT8a_goXcW3rCu6RIdgqmP78I_3xI-pvleumFyGLeo$ the Markdown links check is failing because there is a reference to Debian 9 guide. https://www.linode.com/docs/guides/list-tables-in-mysql-and-mariadb/ This links in the Before you Begin section for this test to pass.

— Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/linode/docs/pull/7321*issuecomment-3188122276__;Iw!!GjvTz_vk!UyH7xlcmZeQ5QjV0rEK7Ku5dv-hU7q9RkYkjuQKdaXNAteT8a_goXcW3rCu6RIdgqmP78I_3xI-pvleuxF_Q5zk$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/BTI6GIXXU2YVLIQFMRQZDCT3NRXS3AVCNFSM6AAAAACDXDFUPGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTCOBYGEZDEMRXGY__;!!GjvTz_vk!UyH7xlcmZeQ5QjV0rEK7Ku5dv-hU7q9RkYkjuQKdaXNAteT8a_goXcW3rCu6RIdgqmP78I_3xI-pvleu07_kIwU$. You are receiving this because you were mentioned.Message ID: @.@.>>

DHBR2 avatar Aug 14 '25 16:08 DHBR2

So, are you saying that I have to update a different document that is not part of the assigned changes in order to make this PR successful? Let me introduce myself - I work for Andy Stevens as a Technical Writer and am trying to learn the process for getting PRs to pass all checks. The Issue: * The PR is failing a "Markdown links check" * The failure is due to a broken or outdated link to a Debian 9 guide (https://www.linode.com/docs/guides/list-tables-in-mysql-and-mariadb/) * This link exists in the "Before you Begin" section of a document I didn’t modify * Even though this broken link isn't part of my changes, it's causing my PR to fail I’m happy to fix it and add it to the PR, but I wanted to understand if there is an established process for handling this type of situation? My original assignment was to update the one document’s mention of Debian 9 and make relevant changes to the subject matter - no other documentation updates were mentioned. Is fixing unrelated broken links typically the contributor’s responsibility, or is there another way to handle this? From: Rajakavitha Kodhandapani @.> Sent: Thursday, August 14, 2025 4:32 AM To: linode/docs @.> Cc: Hoober, Dianna @.>; Mention @.> Subject: Re: [linode/docs] Clean cherry-pick: MariaDB guide update for Debian 12 (PR #7321) [https://avatars.githubusercontent.com/u/37059749?s=20&v=4]Rajakavitha1 left a comment (linode/docs#7321)https://urldefense.com/v3/__https:/github.com/linode/docs/pull/7321*issuecomment-3188122276__;Iw!!GjvTz_vk!UyH7xlcmZeQ5QjV0rEK7Ku5dv-hU7q9RkYkjuQKdaXNAteT8a_goXcW3rCu6RIdgqmP78I_3xI-pvleuxF_Q5zk$ Hi @DHBR2https://urldefense.com/v3/__https:/github.com/DHBR2__;!!GjvTz_vk!UyH7xlcmZeQ5QjV0rEK7Ku5dv-hU7q9RkYkjuQKdaXNAteT8a_goXcW3rCu6RIdgqmP78I_3xI-pvleumFyGLeo$ the Markdown links check is failing because there is a reference to Debian 9 guide. https://www.linode.com/docs/guides/list-tables-in-mysql-and-mariadb/ This links in the Before you Begin section for this test to pass. — Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/linode/docs/pull/7321*issuecomment-3188122276__;Iw!!GjvTz_vk!UyH7xlcmZeQ5QjV0rEK7Ku5dv-hU7q9RkYkjuQKdaXNAteT8a_goXcW3rCu6RIdgqmP78I_3xI-pvleuxF_Q5zk$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/BTI6GIXXU2YVLIQFMRQZDCT3NRXS3AVCNFSM6AAAAACDXDFUPGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTCOBYGEZDEMRXGY__;!!GjvTz_vk!UyH7xlcmZeQ5QjV0rEK7Ku5dv-hU7q9RkYkjuQKdaXNAteT8a_goXcW3rCu6RIdgqmP78I_3xI-pvleu07_kIwU$. You are receiving this because you were mentioned.Message ID: @.@.>>

Hi @DHBR2 Just made the updates to the format you can see the format changes in the commits. I also made changes in the related files so that the CI/CD tests are successful.

Rajakavitha1 avatar Aug 15 '25 06:08 Rajakavitha1

LGTM

Screenshot 2025-08-15 at 11 35 12 AM

Rajakavitha1 avatar Aug 15 '25 06:08 Rajakavitha1

LGTM

Screenshot 2025-08-15 at 11 35 12 AM

Hi @andystevensname I have tested and validated please review the changes. Also not sure if this file is required. docs/guides/databases/mariadb/how-to-install-mariadb-on-debian-12/Installing_MariaDB_on_Debian9.png. Please let me know know if this can be deleted for this PR.

Rajakavitha1 avatar Aug 15 '25 06:08 Rajakavitha1