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.

Sign up for GitHub

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

Jump to bottom

fix(index): switch inclusive language to warning #77

Merged
merged 1 commit into from
May 26, 2021
Merged

fix(index): switch inclusive language to warning #77

merged 1 commit into from
May 26, 2021

Conversation

nellyk
Copy link
Contributor

@nellyk nellyk commented May 26, 2021

Change from error to warning to remove breaking change

Description

Please make sure that the PR fulfills these requirements

  • I checked that there aren't other open Pull Requests for the same update/change.
  • My code follows the code style of this project.
  • My change requires a change to the documentation and I have updated the documentation accordingly.
  • Rule changes includes a comment to describe the reasoning behind the change.
  • PR contains a single rule change.

Motivation and Context

How Has This Been Tested?

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation (adding or updating documentation)
  • Dependency update

@nellyk nellyk requested a review from a team as a code owner May 26, 2021 17:15
@smackfu smackfu requested a review from a team May 26, 2021 17:17
@nellyk nellyk merged commit b76f28d into main May 26, 2021
@nellyk nellyk deleted the fix/inclusive-language branch May 26, 2021 17:52
oneamexbot added a commit that referenced this pull request May 26, 2021
@oneamexbot
chore(release): 14.2.1 [skip ci]
f0824a7 
## [14.2.1](v14.2.0...v14.2.1) (2021-05-26)

### Bug Fixes

* **index:** switch inclusive language to warning ([#77](#77)) ([b76f28d](b76f28d))
@oneamexbot
Copy link
Contributor

🎉 This PR is included in version 14.2.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JAdshead JAdshead JAdshead approved these changes

@smackfu smackfu smackfu approved these changes

@marcusrbrown marcusrbrown marcusrbrown approved these changes

Assignees
No one assigned
Labels
released
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@nellyk @oneamexbot @marcusrbrown @smackfu @JAdshead