Skip to content
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

(Content) Top dev blog post #5

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

OumarYanni
Copy link
Contributor

Hi Thomas,

Question: From the readme, it looks like you have to PR directly to the original NX repository (coz we are intern) ? If not, if it's the PR option from a fork from my repo then, please confirm so I can fix it.

As for the blog post, this is the first version. I plan to go through it again to fine-tune the details and complete the tools I've thought of. If it's a lot you'll let me know, also if you'd like me to change the tone, go into more depth or delete something.

Looking forward to your first feedback.

@tdimnet tdimnet self-requested a review August 11, 2024 17:03
Copy link
Member

@tdimnet tdimnet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Oumar,

Thank you for submitting your first blog post!

Could you add a hero image? I generate most of them with GPT-4. It has to be on relationship with the article. You could for example use that prompt: "Des personnes sur un podium, format large, pixel art". Once you generate the image, you can add it inside the public folder

About the package-lock.json, this is strange that it changed before my branch and yours. Could you tell me which version of node do you use?

Use the command on your terminal:

node -v

As a general feedback, I did like what you wrote. I think your first blog post is good. I gave you comments and suggestions to make it better. Don't hesitate if you need help or more information.

Thanks again,
Tom.

src/pages/blog/ce-qui-caractérise-le-top-20%-des-devs.md Outdated Show resolved Hide resolved
src/pages/blog/ce-qui-caractérise-le-top-20%-des-devs.md Outdated Show resolved Hide resolved
src/pages/blog/ce-qui-caractérise-le-top-20%-des-devs.md Outdated Show resolved Hide resolved
src/pages/blog/ce-qui-caractérise-le-top-20%-des-devs.md Outdated Show resolved Hide resolved
src/pages/blog/ce-qui-caractérise-le-top-20%-des-devs.md Outdated Show resolved Hide resolved
src/pages/blog/ce-qui-caractérise-le-top-20%-des-devs.md Outdated Show resolved Hide resolved
src/pages/blog/ce-qui-caractérise-le-top-20%-des-devs.md Outdated Show resolved Hide resolved

Comme moi, vous voulez surement du concret, quelque chose à se mettre sous la dent (et pas n’importe quelle dent, hein, celle d’un dev : on en fait pas des plus acérés). Je vous ai donc préparer deux outils que vous pouvez mettre en pratique dès maintenant.

### Outil 1 : je pense à un formulaire Tally
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you have ressources to share? You don't have to explain everything, especialy if someone else do it better :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll take the time to add more resources tomorrow as it's the end of the day.

One question I'm asking myself is whether it's necessary to have read the whole resource or just an overview, or whether a book that's often recommended but hasn't been read can do the trick.
For my part, the majority of this article is a digest of what I've learned so far and the AI to help me complete it and make it more audible to the majority. I'd love to know the best way to deal with this.

@OumarYanni OumarYanni force-pushed the content_top-dev-blog-post branch from a3f8c98 to 0537c7c Compare August 15, 2024 15:04
@OumarYanni
Copy link
Contributor Author

About the package-lock.json, this is strange that it changed before my branch and yours. Could you tell me which version of node do you use?

I was using node 18 and npm less than 10. I took the time to update them and make sure that my local "package-lock.json" is the same as the one in the original repository by applying the "git checkout -- package-lock.json" command on the (main) and (content_top-dev-blog-post) branches. Let me know if you see a new improvement to integrate.

@OumarYanni
Copy link
Contributor Author

Could you add a hero image? I generate most of them with GPT-4. It has to be on relationship with the article. You could for example use that prompt: "Des personnes sur un podium, format large, pixel art". Once you generate the image, you can add it inside the public folder

It's done, but I didn't know which of the images you'd prefer, so I'll share the others that stood out, below:

_a1798ae8-6cd4-4e58-92a6-c2066087ed31
des-personnes-sur-un-podium-format-large-pixel-art
_4ea62efd-979e-4d3f-aa91-6ea3cc7bc1ac

@tdimnet tdimnet force-pushed the content_top-dev-blog-post branch from 049f3a5 to 1665f11 Compare August 19, 2024 16:46
@tdimnet tdimnet changed the title First version of content_top-dev-blog-post (Content) Top dev blog post Aug 23, 2024
@tdimnet tdimnet changed the base branch from main to feat_new-home September 27, 2024 06:52
Base automatically changed from feat_new-home to main September 27, 2024 17:25
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.

2 participants