-
-
Notifications
You must be signed in to change notification settings - Fork 42
Updated readme with updated file structure #102
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
Conversation
Updated the image link for the Library File Structure.
Updated image link for Library File Structure in README.
Reviewer's guide (collapsed on small PRs)Reviewer's GuideUpdates the README’s library file structure illustration to use a new, embedded image reference that reflects the latest project layout. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey - I've found 1 issue, and left some high level feedback:
- The updated README image link mixes Markdown and HTML (
![... ]<img ... />); choose a single format (Markdown or<img>alone) so the image renders correctly. - If you stick with the
<img>tag, consider keeping the previous descriptive alt text (e.g.,alt="Library File Structure") for accessibility and consistency with the prior README.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- The updated README image link mixes Markdown and HTML (`![... ]<img ... />`); choose a single format (Markdown or `<img>` alone) so the image renders correctly.
- If you stick with the `<img>` tag, consider keeping the previous descriptive alt text (e.g., `alt="Library File Structure"`) for accessibility and consistency with the prior README.
## Individual Comments
### Comment 1
<location> `README.md:103` </location>
<code_context>
<br>
-
+![Library File Structure]<img width="1800" height="1400" alt="easyPythonpi_package_layout_final" src="https://github.com/user-attachments/assets/f5a42d57-0385-4ffe-b9e5-0fd1e80a168a" />
<br>
</code_context>
<issue_to_address>
**issue:** The combined Markdown and HTML image syntax here is likely malformed and may not render as intended.
This combines Markdown (`![Library File Structure]`) with an HTML `<img>` tag in a way most renderers won’t treat as a valid image, so the alt text may show up literally. Please use either a proper Markdown image with URL, or a standalone HTML `<img>` tag, but not both together on the same line.
</issue_to_address>Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
|
hi @extinctsion , please review! |
|
|
hi @extinctsion , please review! |
|
Sure. Please give me some time as I'm a bit busy with other work. I'll surely review the PR. |
extinctsion
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good



Updated README.md to reflect the latest project structure.
Summary by Sourcery
Documentation: