bpo-10379: deprecate locale.format in lieu of locale.format_string by plusminushalf · Pull Request #259 · python/cpython · GitHub

>>> import locale;
>>> print(locale.format('%.0f KB', 100))
100 KB
>>> locale.format.__doc__
'Deprecated, use format_string instead.'

Hello, and thanks for your contribution!

I'm a bot set up to make sure that the project can legally accept your contribution by verifying you have signed the PSF contributor agreement (CLA).

Unfortunately our records indicate you have not signed the CLA. For legal reasons we need you to sign this before we can look at your contribution. Please follow these steps to rectify the issue:

Sign the PSF contributor agreement
Wait at least one US business day and then check "Your Details" on bugs.python.org to see if your account has been marked as having signed the CLA (the delay is due to a person having to manually check your signed CLA)
Reply here saying you have completed the above steps

Thanks again to your contribution and we look forward to looking at it!

Thanks for working on this. Doc changes are also needed.

As well as tests for the deprecation message.

Somehow I hadn't noticed that 'monetary' wasn't supported by format_string. So that is actually an enhancement, in addition to the deprecation, and needs to be documented as such (versionchanged, whats new entry). Also, format itself shouldn't be changed, just deprecated (that is, the deprecation message code added, but nothing else in it changed).

@bitdancer I don't think to keep locale.format as it is and just displaying deprecated warning would be of any use. I think it is safe to call locale.format_string internally.

The description of the monetary parameter should be added to the regular docs, and the version changed phrase should just say "the monetary keyword parameter was added"

That's the way we normally do deprecations: leave the existing code in place (so its behavior doesn't change) but point people to the preferred solution. It would be acceptable if there were no behavior changes, but the changed tests prove that there are.

Thanks. Almost there :)

It would also be good to have an entry for the What's New document for 3.7 for the addition of the monetary parameter to format_string, as well as the deprecation of format. I haven't looked; if the 3.7 what's new hasn't been organized yet, just stick in a placeholder sentence with a reference to the bpo issue number.

We still need the description of the monetary keyword (presumably copy and pasted from the existing format entry) in the description of the format_string function.

In fact, what we really want to do is copy most of the 'format' description into format_string, since it seems wrong somehow to refer to the docs of a deprecated from from the preferred function.

You can use assertWarns here instead.

I don't think the 'replaces' sentence is needed. The deprecated on format covers that.

Fixed recommended CR changes

Thanks for sticking with this :)

Missing indentation.

format hasn't been removed, so the deprecation should go in the deprecation section. The feature addition of the monetary keyword should go in the 'improved modules' section.

Codecov Report

Merging #259 into master will decrease coverage by -1.01%.
The diff coverage is 71.42%.

@@            Coverage Diff             @@
##           master     #259      +/-   ##
==========================================
- Coverage   83.38%   82.38%   -1.01%     
==========================================
  Files        1367     1428      +61     
  Lines      344897   351221    +6324     
==========================================
+ Hits       287596   289342    +1746     
- Misses      57301    61879    +4578

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ae160bb...6097e47. Read the comment docs.

There should be a subsection header before this line for the locale module (the modules with enhancements are listed in this section in alphabetical order).

@bitdancer is it good to go now?

Yes, I think so. I'd like to do a final overall review before I approve it, though. With any luck I'll do that tonight.

Thanks! :)

Hey, @bitdancer have you reviewed this?

I did...and I feel bad that I have some more changes to suggest, so I was going to make them myself and submit it to the PR, but I haven't actually figured out how to do that yet.

@bitdancer well I think you can suggest me the changes I'll do my best to figure them out.

So looking at this again, I think what we should do is move 'format' under 'format_string', and change its body to say "works like format_string but only accepts a single format specification. That followed by the deprecation notice will make it a lot less likely anyone will use it :)

Small nit: our convention appears to be to put the Contributed by as part of the paragraph when there's only one paragraph. Ends up formatted the same in html, but might as well make the ReST consistent as well.

I think this should read "This method will be removed in a future version of Python."

As long as we are editing this, I think there should be a blank line after the first sentence, before the 'Grouping' sentence.

Since we know there's only one format string here, I think this should be a call to _format.

As above, I think this should be _format.

Refactoring Doc, moving format doc below format_string, this is to make sure people have lesser probability of using format in future. Also using _format method inside locale module, since we know there is only one format function that is _format

There was no need to move format in the source file, but it doesn't hurt, either.

Hey @bitdancer is there anything left to get this merged?

Just adding the Misc/NEWS entry. I'm not sure what status is on how to do that currently, but I think it is still manual edit of that file. I haven't had time yet to learn how to update a PR before merge, so if you add the entry, I'll hit merge :)

Google says it's spelled 'lieu'. Also the entry should note that the monetary argument was added to format_string. So, how about "locale.format_string now supports the 'monetary' keyword argument, and locale.format is deprecated."

Update the Stackless patches for readthedocs.org based on the documentation. This simplified configuration is still untested.

Well, the previous commit didn't build on readthedocs.org. - Rename .readthedocs.yml to .readthedocs.yaml See https://docs.readthedocs.io/en/stable/config-file/index.html - Add a requirements file Doc/slp_readthedocs_requirements.txt and require the same packages as specified in Makefile. Already set the Sphinx version to 2.0.1, because upstream commit 7d23dbe

Set the css-file to "pydoctheme.css".

Remove the explicit specification of the css-file for readthedocs.org. It is not needed any more.

bpo-10379: deprecate locale.format in lue of locale.format_string

d0f9863

the-knights-who-say-ni added the CLA not signed label Feb 23, 2017

Mariatta removed the CLA not signed label Feb 24, 2017

the-knights-who-say-ni added the CLA signed label Feb 24, 2017

Adding tests and documentation changes

820e289

Adding version changed info for locale.format

4cfd737

bitdancer requested changes Feb 24, 2017

View reviewed changes

Deprecating but not changing the function defination

a08419f

bitdancer requested changes Feb 24, 2017

View reviewed changes

Adding documentation inside what's new in 3.7

4e24a3f

Fixed recommended CR changes

plusminushalf force-pushed the fix-bpo-10379 branch from 052f6bc to 4e24a3f Compare February 25, 2017 15:07

bitdancer requested changes Feb 26, 2017

View reviewed changes

Adding whatsnew documentation in proper segments

6097e47

plusminushalf added 2 commits February 27, 2017 19:30

Merge branch 'master' of github.com:python/cpython into fix-bpo-10379

04577c2

fixing documentation bug

2039406

bitdancer requested changes Feb 27, 2017

View reviewed changes

Adding locale subsection

5b100b8

Merge branch 'master' of github.com:python/cpython into fix-bpo-10379

041349f

bitdancer reviewed Mar 17, 2017

View reviewed changes

bitdancer approved these changes Mar 20, 2017

View reviewed changes

Adding entry to Misc/NEWS about changes

8190fa9

bitdancer requested changes Mar 28, 2017

View reviewed changes

plusminushalf changed the title bpo-10379: deprecate locale.format in lue of locale.format_string bpo-10379: deprecate locale.format in lieu of locale.format_string Mar 28, 2017

CR fixes

7e0bfcf

bitdancer approved these changes Mar 28, 2017

View reviewed changes

bitdancer merged commit 1cf93a7 into python:master Mar 28, 2017

akruis added a commit to akruis/cpython that referenced this pull request Aug 8, 2021

Stackless issue python#259 : Update Doc/conf.py and .readthedocs.yml

b87f882

Update the Stackless patches for readthedocs.org based on the documentation. This simplified configuration is still untested.

akruis added a commit to akruis/cpython that referenced this pull request Aug 8, 2021

Stackless issue python#259 : Update Doc/conf.py

53ca885

Set the css-file to "pydoctheme.css".

akruis added a commit to akruis/cpython that referenced this pull request Aug 8, 2021

Stackless issue python#259 : Update Doc/conf.py

e61be11

Remove the explicit specification of the css-file for readthedocs.org. It is not needed any more.

roe-dl mentioned this pull request Oct 27, 2024

AttributeError: module 'locale' has no attribute 'format'. Did you mean: '_format'? (Python 3.12) poblabs/weewx-belchertown#954

Closed

                  .. versionchanged:: 3.7
-                 Added *monetary*, if  true the conversion uses monetary thousands separator and
-                 grouping strings.
+                 The *monetary* keyword parameter was added.

+                  """Deprecated, use format_string instead."""
+                  warnings.warn(
+                      "This method will be removed in future versions.  "
+                      "Use 'locale.format_string()' instead.",

                   """Formats a string in the same way that the % formatting would use,
                   but takes the current locale into account.
-                  Grouping is applied if the third parameter is true."""
+                  Grouping is applied if the third parameter is true.

               def str(val):
                   """Convert float to string, taking the locale into account."""
-                  return format("%.12g", val)
+                  return format_string("%.12g", val)

                       self._test_format("%-10.f", 4200, grouping=0, out='4200'.ljust(10))
+                  def test_format_deprecation(self):
+                      with warnings.catch_warnings(record=True) as w:

                 function instead.
+              * Deprecated :meth:`format` from :mod:`locale`, use the :meth:`format_string`
+                instead. Added another argument *monetary* in :meth:`format_string` of

               Improved Modules
               ================
+              * Added another argument *monetary* in :meth:`format_string` of :mod:`locale`.

                  For whole format strings, use :func:`format_string`.
+                 .. deprecated:: 3.7
+                    Use :meth:`format_string` instead

+              If *monetary* is true, the conversion uses monetary thousands separator and
+              grouping strings.
+              (Contributed by Garvit in :issue:`10379`.)

                                        "the 'C' locale.")
-                  s = format('%%.%if' % digits, abs(val), grouping, monetary=True)
+                  s = format_string('%%.%if' % digits, abs(val), grouping, monetary=True)

               - bpo-29534: Fixed different behaviour of Decimal.from_float()
                 for _decimal and _pydecimal. Thanks Andrew Nester.
+              - bpo-10379: Deprecate locale.format in lue of locale.format_string

		grouping strings.
		Replaces :meth:`format`.

bpo-10379: deprecate locale.format in lieu of locale.format_string by plusminushalf · Pull Request #259 · python/cpython · GitHub

Uh oh!

Uh oh!

bpo-10379: deprecate locale.format in lieu of locale.format_string - #259

bpo-10379: deprecate locale.format in lieu of locale.format_string#259bitdancer merged 13 commits intopython:masterpython/cpython:masterfrom plusminushalf:fix-bpo-10379plusminushalf/cpython:fix-bpo-10379Copy head branch name to clipboard

Conversation

plusminushalf commented Feb 23, 2017

Uh oh!

the-knights-who-say-ni commented Feb 23, 2017

Uh oh!

bitdancer commented Feb 23, 2017

Uh oh!

bitdancer commented Feb 23, 2017

Uh oh!

bitdancer commented Feb 24, 2017

Uh oh!

plusminushalf commented Feb 24, 2017

Uh oh!

Choose a reason for hiding this comment

Uh oh!

bitdancer commented Feb 24, 2017

Uh oh!

bitdancer left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

bitdancer left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

codecov Bot commented Feb 27, 2017

Codecov Report

Uh oh!

Choose a reason for hiding this comment

Uh oh!

plusminushalf commented Mar 3, 2017

Uh oh!

bitdancer commented Mar 3, 2017

Uh oh!

plusminushalf commented Mar 3, 2017

Uh oh!

plusminushalf commented Mar 17, 2017

Uh oh!

bitdancer commented Mar 17, 2017

Uh oh!

plusminushalf commented Mar 17, 2017

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

bitdancer commented Mar 20, 2017

Uh oh!

plusminushalf commented Mar 27, 2017

Uh oh!

bitdancer commented Mar 27, 2017

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

bpo-10379: deprecate locale.format in lieu of locale.format_string#259
bitdancer merged 13 commits into
python:masterfrom
plusminushalf:fix-bpo-10379