-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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(blog/events): Dublin collab summit 2024 #7215
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
Awesome! Thanks for this. I'll try to flesh out my section in the next day or so.
Co-Authored-By: Jacob Smith <[email protected]>
fbcd43b
to
5f7a00f
Compare
Commit cotent -> content |
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.
Love it :-)
Co-authored-by: Aviv Keller <[email protected]> Signed-off-by: Augustin Mauroy <[email protected]>
Co-authored-by: Aviv Keller <[email protected]> Signed-off-by: Augustin Mauroy <[email protected]>
Co-authored-by: Aviv Keller <[email protected]> Signed-off-by: Augustin Mauroy <[email protected]>
BTW, we use squash and merge for pr so just the title is important 😄 |
Joyee's feedback on slack:
and
|
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
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.
normalized the names throughout per discussion, some slight cleanup, and resolved a duplication between the codemod and tooling discussions - thanks for helping put this together
Lighthouse Results
|
@AugustinMauroy removed the personal note via 856f262 in the interest of getting this ready to merge. otherwise we would need to solicit for more opinions. I like the spirit but agree it's a good sentiment to communicate personally on a blog or on social media, perhaps while linking to this report. |
Lgtm ! I am fine with it like this thanks Brian for giving energy in this |
|
||
#### Q1: What words or phrases would you use to describe the Node.js collaborator culture? | ||
|
||
- **Silos**: Some attendees felt that the community is divided into silos, with different groups working independently. |
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.
I see that these explanations that were only added in based on personal understanding after the summit was still published. Personally (as one of the only two women, and the only female collaborator in the room) I feel very sad that the report pretended that we had some in-depth discussions about diversity when we in fact, did not, in that mere 30 minutes :(
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.
😅
Feel free to open a pr which changes the message.
Maybe review this part with Robin?
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.
I am too tired to edit it and I think the issue about diversity is now self-explanatory in this PR if it the question about the truthfulness of it from the only female collaborator there was simply ignored, and we needed the only two women in the room to go extra length to make sure that it is reflected truthfully in the report instead of being painted over. So leaving it as-is gives me a pointer for the future as well.
Was this maybe merged too early? Again, it is not up to us (website team) to write content, so I would have waited till everyone in the room was happy with the content 😅 |
Apologies. I should have reached for more consensus. My thought process at the time was a PR open for almost 2 weeks with the temporal nature of the summit content only growing. It's not defensible on those merits alone but honestly thought the collaborator input from non-website maintainers was fully resolved. I see now that was untrue and will work to be more careful in the future. |
All good! Technically speaking all requirements were correct and I also didn't say anything earlier (was so busy) -- but this sort of content is non-trivial and must be put in a way that effectively represents the storylines of the events. No harm was done, happens, we learn. Let's just next time triple check and give proper time to all people involved on talks to consent with our narrative. That's why IMO these articles must be impartial and not from someone's PoV :) |
(Also no rush to put this content out, it's not like people are eager to see this content, the only time requirement is the one we put on ourselves here lol) |
Description
Writting a blog post to summarize the summit.
The aim is to enable people who are not involved in the project to find out what's going on. And to make this post familiar in the way it is written.
Check List
npm run format
to ensure the code follows the style guide.npm run test
to check if all tests are passing.npx turbo build
to check if the website builds without errors.