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

Add Sports namespace #1538

Merged
merged 6 commits into from Feb 14, 2019
Merged

Add Sports namespace #1538

merged 6 commits into from Feb 14, 2019

Conversation

vbrazo
Copy link
Member

@vbrazo vbrazo commented Feb 14, 2019

Checklist

  • Deprecate ::Football
  • Separate v1.9.3 deprecations from current version deprecations
  • Add missing Faker::Construction documentation

To understand more about deprecation and namespaces: #1318

@vbrazo vbrazo self-assigned this Feb 14, 2019
@vbrazo vbrazo changed the title Add sports namespace Add Sports namespace Feb 14, 2019
@ecbrodie
Copy link
Contributor

@vbrazo should the unreleased README and. CHANGELOG be mentioned in the contributing guide? That way, community contributors can make those changes themselves, rather than the core maintainers doing it as a chore.

@vbrazo
Copy link
Member Author

vbrazo commented Feb 14, 2019

I think it's a good idea, but some contributors won't change these files because they're lazy or busy.

Notice that I moved doc/sports/basketball.md to doc/unreleased/sports/baskeball.md because users usually go inside the doc folder and they get confused with the features that are released in the current version/v1.9.3 versus the ones that are only on master. When we release a new version, we just move the unreleased stuff to the main folder.

@vbrazo vbrazo merged commit acd8a71 into master Feb 14, 2019
@vbrazo vbrazo deleted the add-sports-namespace branch February 14, 2019 13:49
davidmorton0 pushed a commit to davidmorton0/faker that referenced this pull request Jul 12, 2021
* Update unreleased/docs and move sport namespace to unreleased folder

* Deprecate ::Football

* Separate v1.9.3 deprecations from current version deprecaions

* Update Faker::Construction documentation

* Update CHANGELOG

* Update CHANGELOG
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants