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

PR#76 is merged to master but not released? #85

Closed
thegoatherder opened this issue Sep 26, 2022 · 1 comment
Closed

PR#76 is merged to master but not released? #85

thegoatherder opened this issue Sep 26, 2022 · 1 comment

Comments

@thegoatherder
Copy link

@domharrington thanks for this great repo.

I see PR #76 was merged to master on 21 Mar 2022... however the last release is v4.0.4 made on 5 Jan 2021.
Can we have a v4.1 release to include these changes please? Without it the README documentation is wrong. The new tag option is included on the docs, but breaks when used in my code:

(node:306360) UnhandledPromiseRejectionWarning: Error: Unknown field: tag
    at F:\DNV\Code\node_modules\gitlog\dist\gitlog.cjs.development.js:166:15
    at Array.forEach (<anonymous>)
    at createCommandArguments (F:\DNV\Code\node_modules\gitlog\dist\gitlog.cjs.development.js:164:20)
    at gitlog (F:\DNV\Code\node_modules\gitlog\dist\gitlog.cjs.development.js:215:26)   
    at process (F:\DNV\Code\log-gen.js:49:25)
    at Object.<anonymous> (F:\DNV\Code\log-gen.js:71:1)
    at Module._compile (internal/modules/cjs/loader.js:1085:14)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1114:10)
    at Module.load (internal/modules/cjs/loader.js:950:32)
    at Function.Module._load (internal/modules/cjs/loader.js:790:12)
@domharrington
Copy link
Owner

Hey! I think this got released in v4.0.8 - can you confirm? @hipstersmoothie is the primary maintainer on this repo 🙏

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

No branches or pull requests

2 participants