Skip to content

/ hanami.github.io

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

Jump to bottom

Update docs #262

Merged
merged 1 commit into from Jan 21, 2017
Merged

Update docs #262

merged 1 commit into from Jan 21, 2017

Conversation

@mgrachev
Copy link
Contributor

@mgrachev mgrachev commented Jan 8, 2017

No description provided.

@mgrachev
Update docs
4000102
@davydovanton
davydovanton reviewed Jan 8, 2017
View changes
source/guides/architecture/overview.md
├── config
│   └── routes.rb
├── controllers
├── public

This comment has been minimized.

Sign in to view
@davydovanton

davydovanton Jan 8, 2017
Member

I think we should retain this folder as empty

This comment has been minimized.

Sign in to view
@mgrachev

mgrachev Jan 8, 2017
Author Contributor

This folder doesn't exist when you create a new project

This comment has been minimized.

Sign in to view
@AlfonsoUceda

AlfonsoUceda Jan 21, 2017 •
edited
Member

@mgrachev yes, it's created

This comment has been minimized.

Sign in to view
@AlfonsoUceda

AlfonsoUceda Jan 21, 2017
Member

@mgrachev forget what I said, it exists in the root of the project haha :)
@AlfonsoUceda
Copy link
Member

@AlfonsoUceda AlfonsoUceda commented Jan 21, 2017

LGTM 👍
@AlfonsoUceda AlfonsoUceda self-assigned this Jan 21, 2017
@AlfonsoUceda AlfonsoUceda merged commit 6373389 into hanami:build Jan 21, 2017
@mgrachev mgrachev deleted the mgrachev:update-docs branch Jan 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AlfonsoUceda AlfonsoUceda

@davydovanton davydovanton

Assignees

@AlfonsoUceda AlfonsoUceda

Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
@mgrachev @AlfonsoUceda @davydovanton