Skip to content

Conversation

@dylanscott
Copy link
Contributor

Hi - really appreciate this package. I was wondering if you'd be open to specifying the package licenses directly in the package.json in addition to the LICENSE file. The reason being that we use the dependency review GitHub action and its allowed-licenses check gets confused by the "SEE LICENSE IN LICENSE" value. We can add exclusions for specific packages, but specifying the license in package.json should allow it to work out-of-the-box with this check.

@dylanscott
Copy link
Contributor Author

I also opened analogous PRs for a couple of downstream packages pulled in here, pyramation/nested-obj#1 and constructive-io/strfy-js#2

reedloden pushed a commit to hex-inc/shared-workflows that referenced this pull request Nov 20, 2025
[Slack
thread](https://hex-tech-hq.slack.com/archives/C031BV503B9/p1763575324824259)

This PR adds temporary license overrides to unblock my use of the
`pgsql-parser` library in hex-inc/hex#37557. All of the packages in
question are MIT licensed, but their package.json `license` field says
"SEE LICENSE IN LICENSE". I've opened PRs on those packages to fix their
license declarations (constructive-io/pgsql-parser#222, pyramation/nested-obj#1
and constructive-io/strfy-js#2).
@pyramation
Copy link
Collaborator

really appreciate you sir! I will be taking care of ALL this today ;) stay tuned!

I'll merge your commits, and import history into a new pnpm workspace, so all the dev tools aren't so scattered, and will share the new repo here later :)

@pyramation pyramation merged commit a147a67 into constructive-io:main Nov 22, 2025
@pyramation
Copy link
Collaborator

Successfully published:
 - nested-obj@0.1.0
 - strfy-js@3.1.0

these two have been updated, and moved to https://github.com/hyperweb-io/dev-utils as indicated on the npm packages, let me know if you need anything else, and you can now make issues for all the utilities in that repo

I'll get on the pgsql-parser ones now...

@pyramation
Copy link
Collaborator

Successfully published:
 - pgsql-deparser@17.12.0
 - pgsql-parser@17.9.0
 - @pgsql/cli@1.32.0
 - pg-proto-parser@1.30.0
 - @pgsql/transform@17.9.0
 - @pgsql/traverse@17.2.0
 - @pgsql/utils@17.8.0

for pgsql-parser and pgsql-deparser — depending on which version of the parser you're using (like 14, 15,16, etc), I still need to publish those... hopefully you're on the latest

@dylanscott
Copy link
Contributor Author

Thanks so much for the quick turnaround! We're on the latest so we're good 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants