djm / remark-shortcodes

A custom Markdown syntax parser for remark that adds support for shortcodes.
MIT License
45 stars 7 forks source link

Does this not work anymore or am I being stupid? #21

Closed chiubaca closed 3 years ago

chiubaca commented 3 years ago

I'm running the example code but both children nodes are returning as type:paragraph ?

image

this is my package,.json if it helps:

{
  "name": "remark-shortcodes-quick-test",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "author": "",
  "license": "ISC",
  "dependencies": {
    "remark-parse": "^9.0.0",
    "remark-shortcodes": "^0.3.1",
    "unified": "^9.2.0"
  }
}
cammanderson commented 3 years ago

I downgraded remark to match the version mentioned in this projects dev dependencies (e.g. ^8.0.0) and it started working

chiubaca commented 3 years ago

Thanks, looks like there was a breaking change to Remark last year which broke a bunch of plugins. This one included.

https://github.com/remarkjs/remark/pull/536

djm commented 3 years ago

Ah, that is a pain.

I don’t actually use this project and have not for about four years, so I can’t give time to do the upgrade myself but I will:

On Mon, 18 Jan 2021 at 08:55, Alex Chiu notifications@github.com wrote:

Thanks, looks like there was a breaking change to Remark last year which broke a bunch of plugins. This one included.

remarkjs/remark#536 https://github.com/remarkjs/remark/pull/536

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/djm/remark-shortcodes/issues/21#issuecomment-762094090, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAVVKCMLKI7OT3CMMSXMPTS2PZRTANCNFSM4VRFE77Q .

-- Darian Moody w: djm.org.uk

djm commented 3 years ago

Ticket to do the work to get this package on remark v13+: https://github.com/djm/remark-shortcodes/issues/19

Closing this ticket for now, so conversation can happen on the existing ticket. Cheers!