-
Notifications
You must be signed in to change notification settings - Fork 267
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
docs/dask.md
: update links due to src/mintpy
structure change
#1308
Conversation
Signed-off-by: Emmanuel Ferdman <[email protected]>
💖 Thanks for opening this pull request! Please check out our contributing guidelines. 💖 |
Reviewer's Guide by SourceryThis PR updates references to MintPy files and fixes a typo. No diagrams generated as the changes look simple and do not need a visual representation. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
PR summaryThis Pull Request updates the documentation to reflect changes in the file locations of MintPy, specifically adjusting the paths in the SuggestionConsider adding a note in the documentation about the reason for the file path changes, which could help users understand the context of the update. Additionally, ensure that all other documentation files referencing these paths are also updated to prevent inconsistencies. Disclaimer: This comment was entirely generated using AI. Be aware that the information provided may be incorrect. Current plan usage: 0.00% Have feedback or need 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 @emmanuel-ferdman - I've reviewed your changes - here's some feedback:
Overall Comments:
- The typo fix changed 'proceesing' to 'proceeding', but it should be 'processing' when discussing parallel computation.
Here's what I looked at during the review
- 🟢 General issues: all looks good
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
@sourcery-ai thank you for the review. I updated the docs to use 'processing'. |
docs/dask.md
: update links due to src/mintpy
structure change
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.
LGTM. Thanks!
🎉 🎉 🎉 Congrats on merging your first pull request! We here at behaviorbot are proud of you! 🎉 🎉 🎉 |
Description of proposed changes
PR Summary
Small PR - Commit 0b0a8eb moved some MintPy files. This PR adjusts sources to changes. It also fixes a typo along the way.
Reminders
Summary by Sourcery
Documentation: