Merged
Conversation
d0e06c8 to
9e832bf
Compare
Member
|
Please make sure you update changes.xml.
Gary
…On Thu, Dec 24, 2020, 04:32 Lukasz Lenart ***@***.***> wrote:
Merged #9 <#9> into master.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#9 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJB6N4A4M6OPEK3HGD7OZ3SWMDDRANCNFSM4SL6FJTA>
.
|
Member
|
Even if this is a test dependency and it won't affect users? |
Member
Absolutely IMO, some users are also developers. Consider that are two categories of "users" here: users of the APIs and contributors to the component. |
Member
|
Is there any guideline what should be included in |
Member
|
I do not create Jira tickets for these kind of changes, I just don't see
the need. If the changes file is updated then the release notes will show
the changes, which is what matters most IMO. We do get a Jira report
generated with the site but that is different and does not end up in the
release notes.
I document all updates. Think of it as transparency, why would you hide the
update? I don't like to tell people to look at the git history when we can
provide this information as part of our workflow.
Gary
…On Fri, Dec 25, 2020, 05:51 Lukasz Lenart ***@***.***> wrote:
Is there any guideline what should be included in changes.xml? does Maven
plugins should also be included? Should I create a JIRA ticket for each?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#9 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJB6NYVPGD5C5CSLEYR5DDSWRVDLANCNFSM4SL6FJTA>
.
|
Member
|
So this means the ultimate source of truth (changes) is |
Member
|
Well, yes, unless you want to go through the git history.
Gary
…On Sat, Dec 26, 2020, 04:32 Lukasz Lenart ***@***.***> wrote:
So this means the ultimate source of truth (changes) is changes.xml?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#9 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJB6N6J6ZNGSZ6EDKNQX3DSWWUS7ANCNFSM4SL6FJTA>
.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Upgrade to JUnit v4.13.1