Merge pull request #21 from TaduJR/merge-upstream-master

fix(src): merge upstream master and fix onClose double-call bug

Author: inimaga

.circleci/config.yml

@@ -0,0 +1,3 @@
+{
+  "extends": ["./node_modules/cz-ls-commits/commitlint"]
+}

.codeclimate.yml

@@ -1,47 +1,41 @@
 module.exports = {
-    env: {
+  env: {
+    browser: true,
+    es2021: true,
+  },
+  extends: ["plugin:react/recommended", "airbnb", "airbnb/hooks"],
+  plugins: ["react", "react-native", "prettier"],
+  overrides: [
+    {
+      env: {
         node: true,
-        jest: true,
-        browser: true,
-        es2021: true,
-    },
-    settings: {
-        react: {
-            version: 'detect',
-        },
-    },
-    extends: ['plugin:react/recommended', 'airbnb', 'airbnb/hooks', 'prettier'],
-    plugins: ['react', 'react-native'],
-    overrides: [
-        {
-            env: {
-                node: true,
-            },
-            files: ['.eslintrc.{js,cjs}'],
-            parserOptions: {
-                sourceType: 'script',
-            },
-        },
-    ],
-    parserOptions: {
-        ecmaVersion: 'latest',
-        sourceType: 'module',
-    },
-    rules: {
-        'import/no-extraneous-dependencies': 0,
-        'lodash/import-scope': 0,
-        'lodash/prefer-lodash-method': 0,
-        'lodash/prefer-noop': 0,
-        'import/prefer-default-export': 0,
-        'react/prop-types': 0,
-        'react/jsx-props-no-spreading': 0,
-        'react/destructuring-assignment': 0,
-        'react/static-property-placement': 0,
-        'react/forbid-prop-types': 0,
-        'react/no-arrow-function-lifecycle': 0,
-        'react/no-unused-class-component-methods': 0,
-        'react/jsx-filename-extension': 0,
-        'arrow-body-style': 0,
-        'default-param-last': 0,
+      },
+      files: [".eslintrc.{js,cjs}"],
+      parserOptions: {
+        sourceType: "script",
+      },
     },
+  ],
+  parserOptions: {
+    ecmaVersion: "latest",
+    sourceType: "module",
+  },
+  rules: {
+    "arrow-body-style": 0,
+    "comma-dangle": 0,
+    "default-param-last": 0,
+    "import/no-extraneous-dependencies": 0,
+    "import/prefer-default-export": 0,
+    "object-curly-newline": 0,
+    "operator-linebreak": 0,
+    "prettier/prettier": "error",
+    "react/destructuring-assignment": 0,
+    "react/forbid-prop-types": 0,
+    "react/jsx-filename-extension": 0,
+    "react/jsx-props-no-spreading": 0,
+    "react/no-arrow-function-lifecycle": 0,
+    "react/no-unused-class-component-methods": 0,
+    "react/prop-types": 0,
+    "react/static-property-placement": 0,
+  },
 };

.commitlintrc.json

@@ -0,0 +1,29 @@
+# To get started with Dependabot version updates, you'll need to specify which
+# package ecosystems to update and where the package manifests are located.
+# Please see the documentation for all configuration options:
+# https://docs.github.com/code-security/dependabot/dependabot-version-updates/configuration-options-for-the-dependabot.yml-file
+
+version: 2
+updates:
+  - package-ecosystem: 'npm'
+    directory: '/'
+    schedule:
+      interval: weekly
+      time: '10:00'
+      timezone: America/New_York
+    commit-message:
+      prefix: 'build'
+      prefix-development: 'build'
+      include: 'scope'
+    groups:
+      prettier:
+        patterns:
+          - prettier
+          - eslint-plugin-prettier
+
+  - package-ecosystem: 'github-actions'
+    directory: '/'
+    schedule:
+      interval: weekly
+      time: '10:00'
+      timezone: America/New_York

.eslintrc.js

@@ -0,0 +1,10 @@
+pull_request_rules:
+  - name: automatic merge for Dependabot pull requests
+    conditions:
+      - author=dependabot[bot]
+      - check-success=build
+      - check-success=commitlint
+      - 'title~=^build\(deps-dev\): bump '
+    actions:
+      merge:
+        method: squash

.github/dependabot.yml

@@ -0,0 +1,25 @@
+name: build
+on: [push, pull_request]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@93cb6efe18208431cddfb8368fd83d5badbf9bfd # v5
+
+      - name: Use Node.js
+        uses: actions/setup-node@53b83947a5a98c8d113130e565377fae1a50d02f # v6
+        with:
+          cache: yarn
+          node-version-file: .nvmrc
+
+      - name: Install dependencies
+        run: yarn
+
+      - name: Run ESLint
+        run: yarn lint
+
+      - name: Run Tests
+        run: yarn test:coverage

.github/mergify.yml

@@ -0,0 +1,39 @@
+name: commitlint
+on: [push, pull_request]
+
+jobs:
+  commitlint:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@93cb6efe18208431cddfb8368fd83d5badbf9bfd # v5
+        with:
+          fetch-depth: 0
+
+      - name: Use Node.js
+        uses: actions/setup-node@53b83947a5a98c8d113130e565377fae1a50d02f # v6
+        with:
+          cache: yarn
+          node-version-file: .nvmrc
+
+      - name: Install dependencies
+        run: yarn
+
+      - name: Lint commit message
+        run: |
+          if [ "${{ github.event_name }}" = "pull_request" ]; then
+            # Only lint first-parent non-merge commits to skip merged upstream history
+            commits=$(git rev-list --no-merges --first-parent \
+              ${{ github.event.pull_request.base.sha }}..${{ github.event.pull_request.head.sha }})
+            if [ -n "$commits" ]; then
+              for sha in $commits; do
+                echo "Linting commit $sha..."
+                yarn commitlint --from="$sha~1" --to="$sha"
+              done
+            else
+              echo "No non-merge commits to lint"
+            fi
+          else
+            yarn commitlint --from=HEAD~1
+          fi

.github/workflows/build.yml

@@ -0,0 +1,16 @@
+name: dependabot
+on: pull_request_target
+
+permissions:
+  pull-requests: write
+  contents: write
+
+jobs:
+  dependabot:
+    runs-on: ubuntu-latest
+    if: github.actor == 'dependabot[bot]' && contains(github.event.pull_request.title, 'deps-dev')
+    steps:
+      - name: Approve Dependabot PR
+        run: gh pr review --approve ${{ github.event.pull_request.html_url }}
+        env:
+          GITHUB_TOKEN: ${{ github.token }}

.github/workflows/commitlint.yml

@@ -4,9 +4,9 @@ jobs:
   autoclose:
     runs-on: ubuntu-latest
     steps:
-    - name: Autoclose issues that did not follow issue template
-      uses: roots/issue-closer@95665902c8ad8808e378371ef57fbedbf99ba744 # v1.1
-      with:
-        repo-token: ${{ secrets.GH_TOKEN }}
-        issue-close-message: "@${issue.user.login} this issue was automatically closed because it did not follow one of the available issue templates. See [here](https://github.com/lawnstarter/react-native-picker-select/issues/new/choose) for available options."
-        issue-pattern: ".*Additional detail*"
+      - name: Autoclose issues that did not follow issue template
+        uses: roots/issue-closer@fec85f2a845cd6cf33eda6e6a4c93026e71f86d1 # v1.2
+        with:
+          repo-token: ${{ secrets.GH_TOKEN }}
+          issue-close-message: "@${issue.user.login} this issue was automatically closed because it did not follow one of the available issue templates. See [here](https://github.com/lawnstarter/react-native-picker-select/issues/new/choose) for available options."
+          issue-pattern: ".*Additional detail*"