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

up to date changelog with fixes #94

Open
wants to merge 70 commits into
base: master
Choose a base branch
from

Conversation

GerHobbelt
Copy link
Contributor

available on npm as @gerhobbelt/changelog

Fixes/addresses these issues & pullreqs:

… output with `-c` on platforms/terminals which are not recognized as color-supporting. (e.g. Windows+msys bash)
…older) due to `const` usage in the code. Discarding antiquated NodeJS support in `changelog` itself now: NodeJS versions 4 and later are supported and CI tested (adjusted TravisCI config accordingly)
…eport anymore: `expect.an.object` is not the correct use: http://chaijs.com/api/bdd/ --> `expect.an('object')`
…uild -2 and before, after having updated to chai 4)
…eport anymore: `expect.an.object` is not the correct use: http://chaijs.com/api/bdd/ --> `expect.an('object')`
# Conflicts:
#	package-lock.json
…m -rf node_modules; rm -f package-lock.json; npm i`
…pdated since 2013) to chalk-supports module (which is linked from rmp website's has-color entry). Added note to README for Windows users who run into issues around console TTY detection failing on several msys/mingw systems: `-c` vs. `--color`.
GerHobbelt and others added 30 commits October 30, 2017 23:58
…le `assert.isObject()` does what it's supposed to do.
… are in sync with npm releases, at least when they have a 'semver' format, so we collect those alongside the ones produced by the npm registry.

This is particularly handy to obtain a complete releases set (CLI: 'all') when a project is worked on via fork + subsequent "scoping" of that package, but that's just one(1) use case; this functionality is also useful for packages which do have releases via git tags, but have not been registered with npm, for example!
- new generated data is appended to the top of an existing file.
- the tool searches the local directory for any file matching the 'changelog*' glob search crriterion and picks the first one it finds. When no file is found, 'CHANGELOG.md' is assumed as a default destination. This behaviour ensures that various incantations of the name do not cause any trouble in `-a`-updating, while any newly created CHANGELOG will have a github-friendly name: "CHANGELOG.md"
… (I don't trust the 'single releae changelog entries only'-behaviour yet...)
Some non-open-source npm packages like
https://www.npmjs.com/package/bitmovin-player contain repository fields
with empty URLs. This commit handles this case which other wise would
throw an error.
…S and active NodeJS versions in package.json (>= 8.0)
…me promise as any positive result.

- fix: typos in test descriptions
- add tests for the fix to prevent regression
…me promise as any positive result.

- fix: typos in test descriptions
- add tests for the fix to prevent regression

# Conflicts:
#	test/changelog.test.js
# Conflicts:
#	package-lock.json
#	package.json
#	test/changelog.test.js
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.

1 participant