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

Real world applications of containerized development #122

Conversation

JohnnieJnr
Copy link
Contributor

@JohnnieJnr JohnnieJnr commented Nov 23, 2024

fixes #7
/claim #7

Writer's Checklist

Writing Structure

  • Use short sentences and paragraphs, and include bucket brigades.
  • Include more than two descriptive H2 headings to organize content.
  • Capitalize headings according to the AP Stylebook (use this tool)
  • Include an introduction with at least two paragraphs before the first H2 section.
  • Use appropriate formatting (bold, italic, underline), notes, quotes, TLDRs, and key points.
  • Incorporate code elements and Markdown format where appropriate.
  • Ensure at least one visual element per “screen” of the article (screenshots, diagrams, tables, graphs, lists, sidenotes, blockquotes).

Fact-Checking

  • Verify all facts and data points included in the article.

Asset Management

  • Save images in the /assets folder.
  • Follow naming conventions: YYYYMMDD_title_of_the_article_IMG_NAME_NO.png.
  • (Optional) Create a GitHub repo for the code referenced in the article and share it.
  • (Optional) Include a link to this Loom video in the PR comments.

Interlinking

Glossary/Definitions

  • Create new definition in /defitnitions folder.

Review and Edit

  • Ensure articles address the needs of the target audience and their search intent.
  • Read the article out loud to catch any awkward phrasing.
  • Run the draft through Grammarly or a similar grammar tool.
  • Double-check adherence to the style guide and repository guidelines.
  • Use the name of the article for the title of the PR.

@mojafa
Copy link
Collaborator

mojafa commented Dec 4, 2024

@JohnnieJnr Looking at this now

@mojafa
Copy link
Collaborator

mojafa commented Dec 4, 2024

@JohnnieJnr I wanted to let you know that I've just pushed some changes to the main branch that fix some linting issues and update the compliance rules. These updates should help you pass the content compliance checks in your current pull request.

When you get a chance, please pull the latest changes from the main branch into your working branch. This should make the integration smoother and help avoid any potential conflicts.

@mojafa
Copy link
Collaborator

mojafa commented Dec 4, 2024

@JohnnieJnr Here's my feedback:

Screenshot 2024-12-05 at 00 36 45

Consistency with Descriptions and Content:

Companies Mentioned: In the article description and metadata, you mention Monzo, Adobe, and Airbnb. However, the actual content focuses on Spotify, GitHub, Shopify, and Netflix. Please ensure that the companies discussed in the article match those mentioned in the description and metadata for consistency.

Inclusion of Required Insights:

Team Workflows and Onboarding: The original issue (Issue #7) specifies the inclusion of "Insights into team workflows, onboarding processes, and long-term benefits." Currently, the article doesn't address these aspects.

Content Enhancements:

Visuals and Definitions: The article would benefit from the inclusion of images, diagrams, or charts to illustrate key points. Additionally, introducing new definitions where appropriate can help clarify complex concepts for readers.
Interlinking Keywords: Please interlink relevant keywords to appropriate URLs within our site to improve SEO and provide readers with additional resources. This includes linking to existing articles or definitions that align with the topics discussed.

Also please go through this hacker news resource again to draw up more inspiration; it's not just about Docker and containerizing, but development workflow enhancements, onboarding processes, and benefits. Cloud Dev Environments use containers, yes, but there's a deeper iinnovation.

Screenshot 2024-12-05 at 00 48 21 Screenshot 2024-12-05 at 00 48 49

Please update the article to address the points mentioned above.

@JohnnieJnr
Copy link
Contributor Author

@mojafa Thanks for your feedback. I'll make the necessary revisions and submit it

@mojafa
Copy link
Collaborator

mojafa commented Dec 6, 2024

@JohnnieJnr PR fails DCO checks.

Also, you've submitted a PR already. So one has to be closed/merged first before we look at another one, equally, if there's someone else working on the same issues, we'll give them higher priority, but of course quality is key.

MEanwhile fix DCO checks, please.

@mojafa
Copy link
Collaborator

mojafa commented Dec 8, 2024

@JohnnieJnr any updates?

@JohnnieJnr
Copy link
Contributor Author

@mojafa working on it

bellatrick and others added 20 commits December 8, 2024 22:59
Signed-off-by: bellatrick <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: Divanshu Grover <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: Divanshu Grover <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: Divanshu Grover <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: Divanshu Grover <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: Divanshu Grover <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: Divanshu Grover <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
bellatrick and others added 26 commits December 8, 2024 23:47
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: Divanshu Grover <[email protected]>
Signed-off-by: Divanshu Grover <[email protected]>
Signed-off-by: Divanshu Grover <[email protected]>
Signed-off-by: Divanshu Grover <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: Divanshu Grover <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: Divanshu Grover <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: Jaafar <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: Jaafar <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
Signed-off-by: JohnnieJnr <[email protected]>
…f github.com:JohnnieJnr/content into Real-World-Applications-of-Containerized-Development
@JohnnieJnr
Copy link
Contributor Author

@mojafa trying to fix the DCO checks is an endless loop of rebasing all commits from the main branch. My working branch is messed up and it's the same for all the PR that I have created

@mojafa
Copy link
Collaborator

mojafa commented Dec 9, 2024

@JohnnieJnr Do this, start from scratch, copy you current contents somewhere, daytona create [your_fork_repo_url], checkout a new branch for say this issue#7, and git add . , git commit -s -m "your changes", git push -u origin feat/article_00

@JohnnieJnr
Copy link
Contributor Author

@mojafa I'm closing this PR and opening another one

@JohnnieJnr JohnnieJnr closed this Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Real-World Applications of Containerized Development: Case Studies and Lessons Learned