Skip to content

Changelog writer. #565

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

Merged
merged 1 commit into from
Nov 23, 2017
Merged

Conversation

diasbruno
Copy link
Collaborator

@diasbruno diasbruno commented Nov 23, 2017

Now we avoid the RCs and Alpha version, so they won't appear on the
log.

Also, now we negate the log entries that are related to releases
using a filter, which is better than exclude the first entry
using Changelog::log_in_between_versions (that way, sometimes we
remove a valid entry).

Changes proposed:

  • Fix changelog writer.

Upgrade Path (for changed or removed APIs):

  • None

Acceptance Checklist:

  • All commits have been squashed to one.
  • The commit message follows the guidelines in CONTRIBUTING.md.
  • Documentation (README.md) and examples have been updated as needed.
  • If this is a code change, a spec testing the functionality has been added.
  • If the commit message has [changed] or [removed], there is an upgrade path above.

@diasbruno diasbruno force-pushed the fix/changelog-writer branch 2 times, most recently from 2c30f60 to 61e9409 Compare November 23, 2017 18:47
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 61e9409 on diasbruno:fix/changelog-writer into ** on reactjs:master**.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 86.265% when pulling 61e9409 on diasbruno:fix/changelog-writer into a296627 on reactjs:master.

@diasbruno diasbruno force-pushed the fix/changelog-writer branch from 61e9409 to f5dd944 Compare November 23, 2017 18:54
now we avoid the RCs and Alpha version, so they won't appear on the
log.

also, now we negate the log entries that are related to releases
using a filter, which is better than exclude the first entry
using Changelog::log_in_between_versions (that way, sometimes we
remove a valid entry).
@diasbruno diasbruno force-pushed the fix/changelog-writer branch from f5dd944 to 92065af Compare November 23, 2017 18:55
@coveralls
Copy link

Coverage Status

Coverage remained the same at 86.265% when pulling 92065af on diasbruno:fix/changelog-writer into a296627 on reactjs:master.

@diasbruno diasbruno merged commit c1e535f into reactjs:master Nov 23, 2017
@coveralls
Copy link

Coverage Status

Coverage remained the same at 86.265% when pulling 92065af on diasbruno:fix/changelog-writer into a296627 on reactjs:master.

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

Successfully merging this pull request may close these issues.

2 participants