-
Notifications
You must be signed in to change notification settings - Fork 3
Develop #193
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
Develop #193
Conversation
✅ Deploy Preview for shakeelmohamed ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Reviewer's GuideThis PR refines site content and documentation by updating availability messaging, enhancing the experience section with a new senior designer role, standardizing conference link labels, and expanding the project TODO checklist. 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.
Pull Request Overview
This pull request updates the portfolio website to reflect current freelance availability and recent work experience. The main changes remove references to seeking full-time employment and add details about a new freelance position at TALISMAN.
- Updated availability status from "freelance and full-time" to "freelance only" across the site
- Added new freelance position at TALISMAN as Freelance Senior Designer
- Updated company name for existing freelance work from "Self-Employed" to "shakeel.design"
Reviewed Changes
Copilot reviewed 5 out of 8 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| todo.md | Added various project tasks and notes, including resume updates and new project ideas |
| src/index.pug | Updated homepage tagline to remove full-time availability |
| src/about/index.pug | Updated about page availability text and added TALISMAN experience |
| docs/index.html | Compiled HTML reflecting homepage tagline changes |
| docs/about/index.html | Compiled HTML reflecting about page changes and new experience entry |
Comments suppressed due to low confidence (1)
src/about/index.pug:1
- [nitpick] The company name 'shakeel.design' should be capitalized consistently with how it appears elsewhere in the codebase or follow standard company name formatting.
---
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
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 there - I've reviewed your changes - here's some feedback:
- Remove the commented-out Pug lines (//-) now that full-time availability is deprecated to avoid cluttering the templates.
- The nested todo.md checklist items have inconsistent indentation—flatten or clearly nest subtasks for better readability.
- You’re duplicating content updates in both docs and src; consider extracting shared fragments or partials to avoid syncing the same text in two places.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Remove the commented-out Pug lines (//-) now that full-time availability is deprecated to avoid cluttering the templates.
- The nested todo.md checklist items have inconsistent indentation—flatten or clearly nest subtasks for better readability.
- You’re duplicating content updates in both docs and src; consider extracting shared fragments or partials to avoid syncing the same text in two places.
## Individual Comments
### Comment 1
<location> `todo.md:25` </location>
<code_context>
## Bio revision
+- [ ] TPS: wristband motion... there is not face between inside/outside oops!
+- [ ] TPS: show motion design system (3x vertical social posts, LT, slates/bumpers) - show the system/grid, adaptive resizing
- [ ] check OG images for projects, then posts, etc. Consider gif/video versions
</code_context>
<issue_to_address>
**issue (typo):** Typo: 'not face' should be 'no face'.
Please update the phrase to 'there is no face between inside/outside oops!' for improved clarity.
```suggestion
- [ ] TPS: wristband motion... there is no face between inside/outside oops!
```
</issue_to_address>Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
|
|
||
| ## Bio revision | ||
|
|
||
| - [ ] TPS: wristband motion... there is not face between inside/outside oops! |
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.
issue (typo): Typo: 'not face' should be 'no face'.
Please update the phrase to 'there is no face between inside/outside oops!' for improved clarity.
| - [ ] TPS: wristband motion... there is not face between inside/outside oops! | |
| - [ ] TPS: wristband motion... there is no face between inside/outside oops! |
Summary by Sourcery
Revise website and resume content by updating professional roles and availability, refining speaking entries, and extending the project todo list with new tasks and research notes
Enhancements:
Documentation: