Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Re-add the Faker::Internet.base64 method #2378

Merged
merged 2 commits into from Sep 23, 2021

Conversation

ashishra0
Copy link
Contributor

@ashishra0 ashishra0 commented Aug 29, 2021

#2375

Description:

This PR re-adds the Faker::Internet.base64 method which was accidentally removed during the conflict resolution in
#1730.

@ashishra0
Copy link
Contributor Author

@psibi please verify. Thanks

Copy link
Member

@psibi psibi left a comment

Choose a reason for hiding this comment

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

LGTM apart from a minor comment.

# #=> "r_hbZ2DSD-ZACzZT"
# @example
# Faker::Internet.base64(length: 4, padding: true, urlsafe: false)
# #=> "x1/R="
Copy link
Member

Choose a reason for hiding this comment

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

Can you add faker.version since the 2.11.0 series (https://github.com/faker-ruby/faker/blob/master/CHANGELOG.md#v2110-2020-03-24) except the latest one where it was removed mistakely ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Like this @faker.version 2.11.0 here?

Copy link
Contributor Author

@ashishra0 ashishra0 Aug 31, 2021

Choose a reason for hiding this comment

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

@psibi added the faker.version do check if it's the appropriate one :)

Copy link
Member

Choose a reason for hiding this comment

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

Can you check if it's available in v2.19.0 ? (If it's not, probably you can add that information in the docs here)

@ashishra0 ashishra0 requested a review from psibi August 31, 2021 12:39
@vbrazo vbrazo merged commit 708f43e into faker-ruby:master Sep 23, 2021
@cloco-ishiguro
Copy link

Can you please update this fix to 2.19.0?

@lostie
Copy link

lostie commented Jan 17, 2022

Hi @vbrazo @connorshea thanks for the great work on the gem. What are your plans to release a new version that includes these changes?

@connorshea
Copy link
Member

I don't have the ability to cut new releases, so it'd be up to whenever @vbrazo has time to run a new release :)

aclemons pushed a commit to aclemons/faker that referenced this pull request Mar 2, 2022
* Re-add the `Faker::Internet.base64` method

* add  since 2.11.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants