Skip to content

Conversation

@chenpoyo0219
Copy link
Contributor

Proposed changes

This pull request addresses incorrect internal link references introduced in the previous PR. The links were pointing to outdated or non-existent sections, and this update ensures they now correctly direct users to the intended content. This fix improves navigation and content integrity across the documentation.

I apologize for overlooking these issues earlier.

Types of changes

What types of changes does your content introduce to this project?
Put an x in the boxes that apply (without "`")

  • Proofreading (fix spelling, grammar, punctuation, accuracy) Proofreading
  • New content (new section, new readme, etc.)
  • Modifications (modifying existing content)
  • Other (please describe)

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. Don't hesitate to ask if you're unsure about any of them. We're here to help! This simply reminds us of what we will look for before merging your code.
Note: If you submit code make sure to test it

  • Tested any code that was added/modified in this PR
  • Added external links
  • Added internal links
  • Added images
  • Added videos
  • Added code snippets

Reviewers:

  • @username

@chenpoyo0219
Copy link
Contributor Author

Hi, after renaming the folder to remove spaces, the CI still fails with the same error.
From the error message:
FileNotFoundError: 'home/runner/work/computer_science/computer_science/Data_Types/readme_tw.md'
It seems the path generated by test_code.py is missing the leading /, so CI looks for:
home/runner/work/...
instead of the correct absolute path:
/home/runner/work/...
This indicates a bug in the path construction logic for .md files in the test script.
The file does exist in the repo, but CI cannot locate it due to the malformed path.
Could you please check this part of the script?
Thanks for your help! (GPT's thought)

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