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

Guides 2.0 - Architecture #98

Open
wants to merge 6 commits into
base: docs/guides-2.0
Choose a base branch
from

Conversation

swilgosz
Copy link
Member

@swilgosz swilgosz commented Dec 21, 2021

Overview

Architecture chapter of the guides 2.0 (#97)

@swilgosz swilgosz self-assigned this Dec 21, 2021
@swilgosz swilgosz mentioned this pull request Dec 21, 2021
19 tasks
@cllns cllns changed the title Docs/guides 2.0 architecture Guides 2.0 - Architecture Jan 7, 2022
content/v2.0/architecture/overview.md Outdated Show resolved Hide resolved
content/v2.0/architecture/overview.md Outdated Show resolved Hide resolved
content/v2.0/architecture/overview.md Outdated Show resolved Hide resolved
content/v2.0/architecture/overview.md Outdated Show resolved Hide resolved
content/v2.0/architecture/other-files.md Outdated Show resolved Hide resolved
@swilgosz swilgosz force-pushed the docs/guides-2.0-architecture branch from f2c9076 to 9de1a82 Compare May 1, 2022 22:31
@swilgosz
Copy link
Member Author

@swilgosz
Copy link
Member Author

@tak1n could you take another look?

Copy link
Member

@tak1n tak1n left a comment

Choose a reason for hiding this comment

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

Left some minor comments, other than that I think it already explains quite well the essential concepts of hanami 2.

content/v2.0/architecture/overview.md Outdated Show resolved Hide resolved
tree -L 1
.
├── Brewfile
├── Brewfile.lock.json
Copy link
Member

Choose a reason for hiding this comment

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

Seems this file isn't existing in a newly created hanami 2 application (currently through the template repo).

Copy link
Member Author

Choose a reason for hiding this comment

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

In beta even more files had been removed. This will need another walkthrough.

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.

2 participants