I don't quite understand the error. If anyone can tell what it means, please let me know. Thanks

Apparently, mkYarnPackage uses IFD to turn package.json into yarn.nix

and i need to use yarn2nix (the binary) to generate a yarn.nix file and then set yarnNix = ./yarn.nix; to skip the IFD step

I marked this as stale due to inactivity. → More info

@7c6f434c Can you please explain why this was merged without meta and properly formatted commit messages?

Wait, license (specified in the data) doesn't get put into meta.license? missed that.

Oh, I didn't know mkYarnPackage also populates meta with data from package.json. However, meta.homepage and meta.maintainers are missing.

I am not strict about homepage even now (and about commit naming if at least some commits are named well enough to be searchable), and I wasn't worrying much about maintainership record two and a half years ago, I admit that.

More constructively and forward-looking: @chessai would you add yourself as a maintainer here?

I am not strict about homepage even now (and about commit naming if at least some commits are named well enough to be searchable), and I wasn't worrying much about maintainership record two and a half years ago, I admit that.

More constructively and forward-looking: @chessai would you add yourself as a maintainer here?

yes

I am not strict about homepage even now (and about commit naming if at least some commits are named well enough to be searchable)

I think we should care about homepages. An easy-to-click link to use during reviews is worth a lot.
Here, the commits should have been squashed and the first one should have mentioned the version. That's our convention outlined in the contributing guidelines. If you want to merge PRs you should be willing to follow those.

This adds the missing meta attributes and adds @chessai as maintainer: #253749