Skip to content
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

Remove trailing space from generated yaml warning message #84

Conversation

piao-mdsol
Copy link
Contributor

@piao-mdsol piao-mdsol commented Aug 1, 2018

Hi there 👋

Noticed a trailing space 👾 is left in the generated yaml warning message. 👇

screenshot 2018-08-01 14 31 16

So take the chance to chop it ✂️ 🙂

FYI @mdsol/team-10

@piao-mdsol
Copy link
Contributor Author

Hmm, not sure about the CI failure, seems not related though 😕 https://travis-ci.org/mdsol/dice_bag/jobs/410648898

image

@piao-mdsol
Copy link
Contributor Author

I see.. we're removing failing ruby 1.9.3 from the CI in https://github.com/mdsol/dice_bag/pull/83/files#diff-354f30a63fb0907d4ad57269548329e3L3

@jfeltesse-mdsol
Copy link
Contributor

since the code has changed, even a TEENY update may be needed.

  • version bump
  • changelog

@piao-mdsol
Copy link
Contributor Author

Bumped and updated 8917c4b 🆙 🎈 (Waiting for #83 to fix the CI, will rebase after that)

@piao-mdsol piao-mdsol force-pushed the feature/remove-trailing-space-from-yaml-warning-message branch from 8ef6b80 to 9fa4ef2 Compare August 2, 2018 04:45
@piao-mdsol
Copy link
Contributor Author

@mdsol/team-10 Rebased and now CI passed, please 🙏

@jfeltesse-mdsol
Copy link
Contributor

wasn't this the PR where you'd update the changelog and do a TEENY bump?

@piao-mdsol
Copy link
Contributor Author

@jfeltesse-mdsol sorry I got completely lost between switching branches 😓 🙇 🤦‍♂️

@jfeltesse-mdsol jfeltesse-mdsol merged commit df1c630 into develop Aug 2, 2018
@jfeltesse-mdsol jfeltesse-mdsol deleted the feature/remove-trailing-space-from-yaml-warning-message branch August 2, 2018 05:40
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