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

referencing VRaptor4 in README #623

Merged
merged 5 commits into from
Jan 4, 2015

Conversation

nykolaslima
Copy link
Contributor

Closes #622

@Turini what do you think, any suggestion?

Latest version
==============
The latest version is [VRaptor 4](https://github.com/caelum/vraptor4)
and it's use should be preferable
Copy link
Member

Choose a reason for hiding this comment

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

we could add "It's documentation can be found at http://vraptor.org"

@Turini
Copy link
Member

Turini commented Dec 30, 2014

it looks nice to me 👍

@nykolaslima
Copy link
Contributor Author

Done :)

@@ -5,6 +5,13 @@ VRaptor 3
A web MVC framework for fast and maintainable development.
Documentation can be found at http://vraptor3.vraptor.org

Latest version
==============
The latest version is [VRaptor 4](https://github.com/caelum/vraptor4)
Copy link
Member

Choose a reason for hiding this comment

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

What you think?

The latest version is available at [VRaptor 4](https://github.com/caelum/vraptor4) (...)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's nice for me! But we keep the and it's use should be preferable.?

Copy link
Member

Choose a reason for hiding this comment

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

if you use available at, you should provide the full link, without labels, otherwise it's strange: available at VRaptor 4

Copy link
Member

Choose a reason for hiding this comment

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

I think so.

@nykolaslima
Copy link
Contributor Author

Done guys! @lucascs @garcia-jj @Turini

@lucascs
Copy link
Member

lucascs commented Jan 4, 2015

Thanks!

lucascs added a commit that referenced this pull request Jan 4, 2015
@lucascs lucascs merged commit db2111f into caelum:master Jan 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Link VRaptor4 in README
4 participants