Skip to content
This repository has been archived by the owner on Mar 8, 2023. It is now read-only.

chore(dependencies): update eslint-config-airbnb-base #774

Merged
merged 4 commits into from
Nov 10, 2021
Merged

Conversation

abdonrd
Copy link
Collaborator

@abdonrd abdonrd commented Nov 9, 2021

Update eslint-config-airbnb-base.

  • Removing use strict because eslint-config-airbnb-base update.

Update ESLint & Prettier.

I had to upgrade from eslint-config-carbon-base to eslint-config-carbon in order to upgrade Prettier to v2 and the ESLint dependencies.

@ibmdotcom-bot
Copy link
Contributor

ibmdotcom-bot commented Nov 9, 2021

@ibmdotcom-bot
Copy link
Contributor

ibmdotcom-bot commented Nov 9, 2021

@ibmdotcom-bot
Copy link
Contributor

ibmdotcom-bot commented Nov 9, 2021

@ibmdotcom-bot
Copy link
Contributor

ibmdotcom-bot commented Nov 9, 2021

@abdonrd abdonrd force-pushed the lint branch 3 times, most recently from b7ed6c9 to 0c82537 Compare November 10, 2021 08:43
@abdonrd abdonrd changed the title chore(dependencies): update eslint & prettier chore(dependencies): update eslint dependencies Nov 10, 2021
@abdonrd abdonrd marked this pull request as ready for review November 10, 2021 08:46
@abdonrd abdonrd requested a review from a team as a code owner November 10, 2021 08:46
@abdonrd abdonrd requested review from photodow, annawen1 and andysherman2121 and removed request for a team November 10, 2021 08:46
@abdonrd abdonrd changed the title chore(dependencies): update eslint dependencies chore(dependencies): update eslint-config-airbnb-base Nov 10, 2021
@abdonrd abdonrd added the Ready to merge Label for the pull requests that are ready to merge label Nov 10, 2021
@abdonrd abdonrd requested a review from jeffchew November 10, 2021 16:08
Copy link
Member

@jeffchew jeffchew left a comment

Choose a reason for hiding this comment

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

LGTM!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Ready to merge Label for the pull requests that are ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants