Skip to content

Fix broken links in the Devguide #681

Closed

Description

@Mariatta

Looks like the Link check step in the CI is broken. It has always been allowed to fail for a long time.
Would be great to fix them all.

Please see the output here:
https://github.com/python/devguide/runs/2463428458?check_suite_focus=true

While looking at the output, I noticed that pypy.org is listed as broken. I think that's the correct url though. I've filed the issue here:

pypy/pypy.org#23
Update: the url seems to work now.

But it would be good to look through the rest and update the urls accordingly.

Activity

  1. za commented on Oct 29, 2021

    @za
    Contributor

    Hi @Mariatta

    I can't see the output anymore:

    The logs for this run have expired and are no longer available.

    Do you mind to re-run/paste/summarize the output?

    Thank you.

  2. MohitMandalia commented on Dec 12, 2021

    @MohitMandalia

    Hi @Mariatta,

    I would like you to check this link from documentation it says website can't be reached , I am not sure what site it points.

    Also this link points to an empty git repo.

    Please let me know how to fix those.

    Thank you.

  3. hugovk commented on Dec 12, 2021

    @hugovk
    Member

    I would like you to check this link from documentation it says website can't be reached , I am not sure what site it points.

    https://t.me/pybr_i18n is a link to a Telegram channel. If you have Telegram installed on your phone, it opens the channel in the app. Tested just now on Android and it works.

  4. MohitMandalia commented on Dec 19, 2021

    @MohitMandalia

    I would like you to check this link from documentation it says website can't be reached , I am not sure what site it points.

    https://t.me/pybr_i18n is a link to a Telegram channel. If you have Telegram installed on your phone, it opens the channel in the app. Tested just now on Android and it works.

    Thanks.

  5. BrianWisniewski commented on Jan 19, 2022

    @BrianWisniewski

    There also appears to be a broken link within both steps 5 and 7 here.

  6. hugovk commented on Jan 20, 2022

    @hugovk
    Member

    https://github.com/python/voters/ is a private repo so only those with access can see it.

    I've created #793 to try and make it more obvious the repo is locked/private.

  7. slateny commented on Apr 14, 2022

    @slateny
    Contributor

    Some more broken links (copy the text then ctrl+f in the page) that I'm not too sure what to do:

    described in the buildbot documentation
    For an example, see the Py_NewRef (not broken, but misaligned - maybe use hash instead of Python version)
    The Tracker-discuss mailing list
    In Python’s Zulip Chat
    Extended bio (for Brett Cannon)

  8. hugovk commented on Apr 14, 2022

    @hugovk
    Member

    described in the buildbot documentation

    Here's an archived version or perhaps you can find something relevant at https://docs.buildbot.net/current/ ?

    For an example, see the Py_NewRef (not broken, but misaligned - maybe use hash instead of Python version)

    Yeah, for these, I recommend fixing the #Lxxx-Lyyy bit, and then press the y key on your keyboard to get a permalink.

    The Tracker-discuss mailing list

    There's an archive but "Discussions about the bug tracker itself", referring to bpo, is irrelevant so I think this can be removed.

    In Python’s Zulip Chat

    I think this can be removed, but @Mariatta can confirm as it referred to her office hour in Zulip.

    (https://devguide.python.org/help) Extended bio (for Brett Cannon)

    Hi @brettcannon, this used to link to https://stackoverflow.com/cv/DrBrettCannon but it's 404. Would you like another link here, or have it removed?

  9. brettcannon commented on Apr 14, 2022

    @brettcannon
    Member

    @hugovk the link can just go.

    But I honestly don't know if that page is even worth keeping since not many people have filled it out and I know some of it is outdated (e.g. @kushaldas has changed employers).

  10. slateny commented on Apr 15, 2022

    @slateny
    Contributor

    Thanks for the links, Couldn't find a page exactly relevant to buildbot so I used the archieve, and I've updated the PR with all but the Zulip. If Zulip's no longer used, then should the entire section on it be removed?

    And as an aside, if it does get removed, it feels that the help page gets a bit bare and a good chunk of content overlaps with the communications page, so maybe some merging can be considered as well?

  11. hugovk commented on Apr 15, 2022

    @hugovk
    Member

    I've never used Zulip, and I understand it's little used. People will have more luck getting help on the other channels so I'd say it makes sense removing the Zulop section.

    The aside:

    There's still quite a bit of stuff on the help page.

    To me the help page, "Where to Get Help", and the communications page, "Following Python’s Development", serve quite different purposes.

  12. ezio-melotti commented on Apr 17, 2022

    @ezio-melotti
    Member

    I merged actions/toolkit#831.
    I was also looking at the the workflow that runs the make linkcheck:

    - name: Link check
    run: make linkcheck
    continue-on-error: true

    In https://github.com/python/devguide/runs/6050460764?check_suite_focus=true everything looks green, but the "Link check" step failed with

     build finished with problems.
    make: *** [Makefile:132: linkcheck] Error 1
    Error: Process completed with exit code 2.
    

    I understand that the step is allowed to fail (because of the continue-on-error), but the failure seems completely hidden unless you open the logs. I found actions/runner#2347 that discusses the issue, but there doesn't seem to be a clean solution -- just some workarounds.

    I was also exploring other ways to report the failure: steps.<step_id>.outcome can be used to obtain the status of the previous step (e.g. the link check), however I wasn't able to find a way to set the status description of the job so that it shows next to the name of the check.

    In the meanwhile I created actions/toolkit#840, and also found actions/toolkit#727 that is related to the continue-on-error

  13. kushaldas commented on Apr 17, 2022

    @kushaldas
    Member

    @hugovk the link can just go.

    But I honestly don't know if that page is even worth keeping since not many people have filled it out and I know some of it is outdated (e.g. @kushaldas has changed employers).

    Wondering if I should create a PR to update properly!!

  14. hugovk commented on Apr 17, 2022

    @hugovk
    Member

    @kushaldas Sure!

    @ezio-melotti I saw your comment at #727 (comment) and replied to it there: #727 (comment)

  15. added a commit that references this issue on Dec 25, 2022
  16. added a commit that references this issue on Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions