Skip to content
This repository has been archived by the owner on May 13, 2020. It is now read-only.

docs-deprecate this package. #3

Merged
merged 2 commits into from
Aug 1, 2018

Conversation

janwijbrand
Copy link
Member

No description provided.

@janwijbrand janwijbrand changed the title docs-deprecated this package. docs-deprecate this package. Aug 1, 2018
Copy link
Member

@jamadden jamadden left a comment

Choose a reason for hiding this comment

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

It would probably be good to have a similar note in CHANGES.rst, something like

- This package is now considered deprecated in favor of ``zope.password``

README.rst Outdated
@@ -2,11 +2,17 @@
z3c.bcrypt
============

.. warning:: z3c.bcrypt has been superseeded with the new version of
`zope.password`_ as it now includes `bcrypt` support based on
the well-maintained `bcrypt`_ library. Please do no longer use
Copy link
Member

Choose a reason for hiding this comment

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

"Please do no longer" is an awkward phrase. Can it just be "Please do not"?

Copy link
Member Author

Choose a reason for hiding this comment

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

fixed

@janwijbrand janwijbrand merged commit 91c2a8e into zopefoundation:master Aug 1, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants