Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

chore: update @cordova/eslint-config@5.0.0 w/ fixes #574

Merged
merged 2 commits into from
Jul 4, 2023
Merged

chore: update @cordova/eslint-config@5.0.0 w/ fixes #574

merged 2 commits into from
Jul 4, 2023

Conversation

erisu
Copy link
Member

@erisu erisu commented Jul 4, 2023

Motivation and Context

Update @cordova/eslint-config dependency to 5.0.0

Description

  • Updated @cordova/eslint-config to 5.0.0
  • Applied automatic and manual fixes.

Testing

npm t

Checklist

  • I've run the tests to see all new and existing tests pass
  • I added automated test coverage as appropriate for this change
  • Commit is prefixed with (platform) if this change only applies to one platform (e.g. (android))
  • If this Pull Request resolves an issue, I linked to the issue in the text above (and used the correct keyword to close issues using keywords)
  • I've updated the documentation if necessary

@erisu erisu requested review from knight9999 and breautek July 4, 2023 12:04
@erisu erisu added this to the 8.0.0 milestone Jul 4, 2023
breautek
breautek approved these changes Jul 4, 2023
View reviewed changes
Copy link
Contributor

@breautek breautek left a comment

Choose a reason for hiding this comment

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

lgtm

@erisu erisu merged commit a017f47 into apache:master Jul 4, 2023
@erisu erisu deleted the chore/eslint-5.0.0-update branch July 4, 2023 16:32
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@breautek breautek breautek approved these changes

@knight9999 knight9999 Awaiting requested review from knight9999

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
8.0.0
Development

Successfully merging this pull request may close these issues.
2 participants
@erisu @breautek