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

Just a documentation adjustment. #929

Merged
merged 2 commits into from
Jan 28, 2015
Merged

Conversation

renanigt
Copy link
Contributor

Just an adjustment to turn the documentation clearest...

@@ -6,7 +6,7 @@

We will call a `Controller` classes that contain the business model of your application. Those are classes commonly called actions or even services by other frameworks, although they don't mean exactly the same.

With VRaptor's jar in your project, create our controllers in order to receive requests and start building your project.
With VRaptor's jar and its dependencies in your project, create our controllers in order to receive requests and start building your project.
Copy link
Member

Choose a reason for hiding this comment

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

It sounds nice!
What do you think about adding a link to dependencies page here?

With VRaptor's jar and [its dependencies](link to dependencies page here) in your project, etc etc

Copy link
Contributor Author

Choose a reason for hiding this comment

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

True. I forgot. 👍
I'll do this now.

@renanigt
Copy link
Contributor Author

Done @Turini. 😃

Turini added a commit that referenced this pull request Jan 28, 2015
improving docs and adding link to dependencies' page
@Turini Turini merged commit 7fe329f into caelum:master Jan 28, 2015
@Turini
Copy link
Member

Turini commented Jan 28, 2015

Thanks, @renanigt!

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