Skip to content

Show all contributors of the upcoming release#1949

Closed
wilzbach wants to merge 1 commit intodlang:masterfrom
wilzbach:list-contributors
Closed

Show all contributors of the upcoming release#1949
wilzbach wants to merge 1 commit intodlang:masterfrom
wilzbach:list-contributors

Conversation

@wilzbach
Copy link
Copy Markdown
Contributor

@wilzbach wilzbach commented Dec 9, 2017

An initial start at showing this information in the generated changelog.

image
...

This is proof of concept as I would prefer to go without the ugly perl replace hack, i.e. by having clearer templates in the changelog generator (dlang/tools#252).
OTOH this is ready now, works and can be shown on the next changelog. We can always improve it once dlang/tools#252 is in.

This currently only adds the list of contributors to the pending changelog file, but as the changelog generation will be changed soon (e.g. #1907), imho it's better to start small.

@dlang-bot
Copy link
Copy Markdown
Contributor

Thanks for your pull request, @wilzbach!

Bugzilla references

Your PR doesn't reference any Bugzilla issue.

If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog.

@wilzbach
Copy link
Copy Markdown
Contributor Author

wilzbach commented Dec 9, 2017

For the script used to query the contributors, see dlang/tools#253

@wilzbach wilzbach force-pushed the list-contributors branch 5 times, most recently from eadb6ca to 91c1031 Compare December 13, 2017 19:34
posix.mak Outdated
[ -f changelog/pending.dd ] || $(STABLE_RDMD) $(TOOLS_DIR)/changed.d \
$(CHANGELOG_VERSION_LATEST) -o changelog/pending.dd --version "${NEXT_VERSION}" \
--date "To be released" --nightly
changelog/pending.dd: $(TOOLS_DIR)/contributors.d $(TOOLS_DIR)/changed.d $(CHANGELOG_FILES) changelog/next-version | ${STABLE_DMD} $(TOOLS_DIR) $(INSTALLER_DIR)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

make: *** No rule to make target 'changelog/2.033', needed by 'changelog/pending.dd'.  Stop.

$(CHANGELOG_FILES) is not suitable as a target/dependency name directly, because it does not contain file extensions (they are being stripped by basename).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh that error is due to Martin renaming it one day ago (-> https://github.com/dlang/dlang.org/pull/1953/files) and myself forgetting about it in the rebase.

@JackStouffer
Copy link
Copy Markdown
Contributor

Not necessary for this PR but it would be awesome to have a section noting any completely new contributors.

@wilzbach
Copy link
Copy Markdown
Contributor Author

Btw if you don't want to have the perl replacements in there for a short while -> #1838 -> dlang/tools#252

@wilzbach
Copy link
Copy Markdown
Contributor Author

Btw if you don't want to have the perl replacements in there for a short while -> #1838 -> dlang/tools#252

-> #1977

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.

4 participants