Skip to content

Conversation

@izeye
Copy link
Contributor

@izeye izeye commented Oct 16, 2019

This PR removes accidental characters in sample's pom.xml as it seems to be introduced accidentally.

@wilkinsona
Copy link
Member

Thanks very much, @izeye. I'm rather amazed that the sample still built with that extra (2) in it.

@wilkinsona wilkinsona added type: task Non user-facing work and removed status: waiting-for-triage Untriaged issue labels Oct 17, 2019
@wilkinsona wilkinsona added this to the 2.0.5.RELEASE milestone Oct 17, 2019
wilkinsona pushed a commit that referenced this pull request Oct 17, 2019
@izeye izeye deleted the patch-3 branch October 17, 2019 11:14
@izeye
Copy link
Contributor Author

izeye commented Oct 17, 2019

@wilkinsona My pleasure 😄 By the way did you find how the builds have passed with them before? I'm curious as I just imagined that the sample hasn't been included in the builds somehow when I was opening this PR in my iPad.

@wilkinsona
Copy link
Member

I haven't figured it out, but I did confirm that the sample builds fine locally with the (2) in there. It's also fine on CI where the samples are built as part of every build. I guess Maven ignores it somehow 🤷‍♂.

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

Labels

type: task Non user-facing work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants