faker icon indicating copy to clipboard operation
faker copied to clipboard

Suggestion: Address provider should generate state (where appropriate)

Open jenstroeger opened this issue 2 years ago • 10 comments

  • Faker version: 18.4.0
  • OS: macOS 13.2.1

Brief summary of the issue goes here.

The standard address.Provider doesn’t have a state() or state_code() method which would be useful (for countries which support that).

Steps to reproduce

n/a

Expected behavior

n/a

Actual behavior

n/a

jenstroeger avatar Apr 12 '23 10:04 jenstroeger

This issue is stale because it has been open for 30 days with no activity.

github-actions[bot] avatar Jul 12 '23 02:07 github-actions[bot]

This issue was closed because it has been inactive for 14 days since being marked as stale.

github-actions[bot] avatar Jul 27 '23 01:07 github-actions[bot]

The related PR https://github.com/joke2k/faker/pull/1864 is still open, and unfortunately I can’t reopen this issue. Having a PR certainly keeps the issue active…

jens-getpassport avatar Aug 07 '23 21:08 jens-getpassport

This issue is stale because it has been open for 30 days with no activity.

github-actions[bot] avatar Nov 13 '23 01:11 github-actions[bot]

🤷🏻‍♂️

jenstroeger avatar Nov 13 '23 02:11 jenstroeger

Do you want to continue the work from the corresponding PR to resolve this issue?

stefan6419846 avatar Nov 13 '23 16:11 stefan6419846

@stefan6419846 You mean reopening #1864 and finishing what @Sbhat92 left unfinished?

jenstroeger avatar Nov 13 '23 19:11 jenstroeger

Re-opening probably does not work, but yes, continuing the work on your own fork to fix the remaining issues seems like seem more suitable approach than fighting against the stale mechanism of the issue itself, given that most(?) of the work already has been done.

stefan6419846 avatar Nov 13 '23 19:11 stefan6419846

Thanks @stefan6419846. I’m not too familiar with the inner workings of Faker at this point, but reviewing the work on PR #1864 it looks like I should follow that pattern? For my use case I’m mostly interested in es_CO…

jenstroeger avatar Dec 09 '23 10:12 jenstroeger

At least I would propose so, as @fcurella as the current maintainer just pointed out the style issues and the odd isinstance check inside the tests.

stefan6419846 avatar Dec 09 '23 11:12 stefan6419846

This issue is stale because it has been open for 30 days with no activity.

github-actions[bot] avatar Mar 09 '24 01:03 github-actions[bot]

This issue was closed because it has been inactive for 14 days since being marked as stale.

github-actions[bot] avatar Mar 23 '24 01:03 github-actions[bot]