Skip to content

Commit

Permalink
Take out documentation from DeprecatedMapper to discourage its usage.
Browse files Browse the repository at this point in the history
  • Loading branch information
@rizwanreza
rizwanreza committed Jun 20, 2010
1 parent 3ec7b1c commit c20cfe4
Showing 1 changed file with 0 additions and 384 deletions.

3 comments on commit c20cfe4

@igouss
Copy link

@igouss igouss commented on c20cfe4 Jun 21, 2010

Choose a reason for hiding this comment

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

Why would you decrease code quality by removing all comments? Just change a comment to tell that this class is deprecated.

@dewski
Copy link
Contributor

@dewski dewski commented on c20cfe4 Jun 21, 2010

Choose a reason for hiding this comment

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

I second igouss.

@rizwanreza
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hey igouss and dewski, The latest Rails version always takes out the example and documentations of anything that's deprecated. I will add the deprecation comment, btw. Thanks.

Please sign in to comment.