search

certifi / certifi.io

Public website for the Certifi Project.
http://certifi.io
58 stars 5 forks source link

fix typos #1

Closed bsdlp closed 10 years ago

bsdlp commented 10 years ago

official mozilla CA cert bundle name pulled from here

sigmavirus24 commented 10 years ago

@fly this cannot be merged because I caught one of your typos before you sent your PR. Can you rebase please?

bsdlp commented 10 years ago

@sigmavirus24 rebased.

sigmavirus24 commented 10 years ago

@kennethreitz @Lukasa what do you think of the second part of the diff? Specifically the change to:

CA Bundle is extracted from the Mozilla Included CA Certs.

I think it might be better if it were:

CA Bundle is extracted from the `Mozilla Included CA Certificate List <https://www.mozilla.org/en-US/about/governance/policies/security-group/certs/included/>`_.
bsdlp commented 10 years ago

@sigmavirus24 agreed.

sigmavirus24 commented 10 years ago

LGTM. Any objections @kennethreitz @Lukasa ?

kennethreitz commented 10 years ago

I'm literally still in the middle of typing this :)