Skip to content

Conversation

@wilzbach
Copy link
Contributor

@wilzbach wilzbach commented Dec 24, 2016

There's still too much confusion, e.g. here

Btw what do you prefer: "Dlang-Bot" or "Dlang Bot"?

@MartinNowak
Copy link
Member

Btw what do you prefer: "Dlang-Bot" or "Dlang Bot"?

Deppenleerzeichen | Leerzeichen in Komposita

Copy link
Member

@MartinNowak MartinNowak left a comment

Choose a reason for hiding this comment

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

Otherwise LGTM.

README.md Outdated
### Bugzilla

Once a PR has been merged (i.e. the commits got pushed to master or stable), the Dlang-Bot
will automatically comment to Bugzilla with the regarding commit:
Copy link
Member

Choose a reason for hiding this comment

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

This is not actually dlang-bot doing that, but Github's Bugzilla integration.
Both do use the same regex to parse commit messages though ;).

Copy link
Member

Choose a reason for hiding this comment

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

Maybe just put that section below the other ones and explain how Issues are referenced separately.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Maybe just put that section below the other ones and explain how Issues are referenced separately.

I moved the Bugzilla below the Trello section, but kept it in the "Issue referencing chapter" to avoid unnecessary confusion for the user.

@wilzbach wilzbach dismissed MartinNowak’s stale review January 7, 2017 16:19

Moved the Bugzilla section below the Trello section

@wilzbach
Copy link
Contributor Author

wilzbach commented Jan 7, 2017

Otherwise LGTM.

Merging as I moved the Bugzilla section below and it's hopefully an improvement over the empty README.

@wilzbach wilzbach merged commit b155530 into dlang:master Jan 7, 2017
@wilzbach wilzbach deleted the add-readme branch January 7, 2017 16:21
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