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

Bump micromatch from 3.1.10 to 4.0.4 #491

Merged
merged 1 commit into from
Mar 26, 2022
Merged

Bump micromatch from 3.1.10 to 4.0.4 #491

merged 1 commit into from
Mar 26, 2022

Conversation

trivikr
Copy link
Contributor

@trivikr trivikr commented Mar 14, 2022

Refs #490

@Daniel15
Copy link
Member

Since this is a major version bump, is the API the same, or could this result in a change in behaviour?

@trivikr
Copy link
Contributor Author

trivikr commented Mar 25, 2022

Since this is a major version bump, is the API the same, or could this result in a change in behaviour?

I'm not sure.

Since jscodeshift is going to be active again as per #482 (comment), we should switch to GitHub Actions and ensure CI is run before merging any existing PRs. WDYT?

@Daniel15
Copy link
Member

we should switch to GitHub Actions and ensure CI is run before merging any existing PRs. WDYT?

This sounds great!

@trivikr
Copy link
Contributor Author

trivikr commented Mar 25, 2022

PR added to enable GitHub Actions in #493

@trivikr
Copy link
Contributor Author

trivikr commented Mar 26, 2022

Since this is a major version bump, is the API the same, or could this result in a change in behaviour?

The only API used in jscodeshift is isMatch(), and it's not changed as per CHANGELOG
https://github.com/micromatch/micromatch/blob/master/CHANGELOG.md#400---2019-03-20

const matched = matchers.length ? mm.isMatch(path, matchers, { dot:true }) : false;

We should switch to GitHub Actions and ensure CI is run before merging any existing PRs.

This PR was rebased after CI was enabled in #493, and CI is successful.

@Daniel15
Copy link
Member

Sounds good to me!

@Daniel15 Daniel15 merged commit 331f80e into facebook:main Mar 26, 2022
@trivikr trivikr deleted the bump-micromatch branch March 26, 2022 02:50
@dartess
Copy link
Contributor

dartess commented Aug 10, 2022

@trivikr npm still has an old version of micromatch:

{
  "name": "jscodeshift",
  "version": "0.13.1",
  "description": "A toolkit for JavaScript codemods",
  "repository": {
    "type": "git",
    "url": "https://github.com/facebook/jscodeshift.git"
  },
  "bugs": "https://github.com/facebook/jscodeshift/issues",
  "main": "index.js",
  "scripts": {
    "prepare": "cp -R src/ dist/",
    "test": "jest --bail",
    "docs": "rm -rf docs && jsdoc -d docs -R README.md src/collections/* src/core.js src/Collection.js"
  },
  "bin": {
    "jscodeshift": "./bin/jscodeshift.js"
  },
  "keywords": [
    "codemod",
    "recast",
    "babel"
  ],
  "author": "Felix Kling",
  "license": "MIT",
  "dependencies": {
    "@babel/core": "^7.13.16",
    "@babel/parser": "^7.13.16",
    "@babel/plugin-proposal-class-properties": "^7.13.0",
    "@babel/plugin-proposal-nullish-coalescing-operator": "^7.13.8",
    "@babel/plugin-proposal-optional-chaining": "^7.13.12",
    "@babel/plugin-transform-modules-commonjs": "^7.13.8",
    "@babel/preset-flow": "^7.13.13",
    "@babel/preset-typescript": "^7.13.0",
    "@babel/register": "^7.13.16",
    "babel-core": "^7.0.0-bridge.0",
    "chalk": "^4.1.2",
    "flow-parser": "0.*",
    "graceful-fs": "^4.2.4",
    "micromatch": "^3.1.10",
    "neo-async": "^2.5.0",
    "node-dir": "^0.1.17",
    "recast": "^0.20.4",
    "temp": "^0.8.4",
    "write-file-atomic": "^2.3.0"
  },
  "peerDependencies": {
    "@babel/preset-env": "^7.1.6"
  },
  "devDependencies": {
    "babel-eslint": "^10.0.1",
    "eslint": "^5.9.0",
    "jest": "^21",
    "jsdoc": "^3.4.0",
    "mkdirp": "^0.5.1"
  },
  "jest": {
    "roots": [
      "src",
      "bin",
      "parser",
      "sample"
    ]
  }
}

Could you post a fresh package version to npm?

@trivikr
Copy link
Contributor Author

trivikr commented Aug 10, 2022

Could you post a fresh package version to npm?

I'm just a contributor, and don't have maintainer privileges.
@ElonVolo would be the lead for this project as per discussions in #482

@Daniel15
Copy link
Member

Daniel15 commented Aug 10, 2022

Could you post a fresh package version to npm?

I can do this today. For now it's still me doing the releases.

@dartess
Copy link
Contributor

dartess commented Sep 2, 2022

It seems that they forgot about the publication of the update again :(

@Daniel15 or @ElonVolo friendly ping

@janpe
Copy link

janpe commented Sep 28, 2022

@Daniel15 @ElonVolo is there a chance of getting this bump in a new release?

@ElonVolo
Copy link
Collaborator

I’m a Facebook employee so I do not have npm publish superpowers. We’ll have to tap @Daniel15 on the shoulder for that 😄

@janpe
Copy link

janpe commented Oct 4, 2022

@ElonVolo do you happen to know if there is someone else who can publish since Daniel doesn't seem to be responsive at the moment?

@Daniel15
Copy link
Member

Daniel15 commented Oct 4, 2022

I'm so sorry about this! Stuff came up in my personal life that meant I couldn't push the package on August 10th. This was on my todo list but I've been so busy with other stuff that I completely forgot about it. :(

I'm working on it right now.

@Daniel15
Copy link
Member

Daniel15 commented Oct 4, 2022

@janpe @dartess @trivikr @ElonVolo I've just published the new package to npm. I'm having trouble pushing the tag to this repo but I'll do that soon too.

@janpe
Copy link

janpe commented Oct 5, 2022

@Daniel15 thank you very much! I hope things are getting better for you. Take care!

@rarenatoe
Copy link

rarenatoe commented Oct 22, 2022

I'm still experimenting this issue despite updating react-native to 0.70.3

└─ my-project@workspace:.
   └─ react-native@npm:0.70.3 [fceeb] (via npm:0.70.3 [fceeb])
      └─ react-native-codegen@npm:0.70.6 (via npm:^0.70.5)
         └─ jscodeshift@npm:0.13.1 [75b54] (via npm:^0.13.1 [75b54])
            └─ micromatch@npm:3.1.10 (via npm:^3.1.10)

@stianjensen
Copy link

I'm still experimenting this issue despite updating react-native to 0.70.3

I created a PR for that now:
facebook/react-native#35356

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.

8 participants