Exclude dependabot from changelog.md check

[kesselb]

Description:

We decided to use https://dependabot.com for dependency updates. dependabot is not able to add a new line to changelog.md. Its required to exclude pull requests from dependabot from changelog.md check.

Improvements

• Early exit when not a pull request
• https://www.shellcheck.net/ is happy

Known Issues:

NONE

[aeimer]

@danielkesselberg should work. I have some little adaption I would like to see fixed. Give me time until tomorrow

[PatrickSkowronek]

We use echo the show what will executed, even when there will nothing to be executed.

[kesselb]

geli/.travis/changelog.sh

Lines 54 to 55 in 60d2e6a

	echo -e "${YELLOW}+ Pull Request but not against develop or master. No update for CHANGELOG.md required."
	exit 0

Default case when a pull request. Every other case is covered.

[aeimer]

I'm on @PatrickSkowronek side, we always print the task we do on the first line. We should with this tradition. Makes sense in the travis log

[PatrickSkowronek]

We have a convention in the shell files, which @danielkesselberg properly not really know? Please add it. The rest looks good, but I would test it locally.

[aeimer]

Please keep the shell code style we have used in all the other files

[aeimer]

I'm on @PatrickSkowronek side, we always print the task we do on the first line. We should with this tradition. Makes sense in the travis log

[aeimer]

Always end with ${NC} if you define a color before

[aeimer]

if you want to use " do it with the whole param: curl --silent "https://raw.githubusercontent.com/geli-lms/geli/${TRAVIS_BRANCH}/CHANGELOG.md"

[coveralls]

Pull Request Test Coverage Report for Build 243

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+1.3%) to 74.841%

---

Totals
Change from base Build 241:	1.3%
Covered Lines:	1748
Relevant Lines:	2236

---

💛 - Coveralls

[kesselb]

@PatrickSkowronek @aeimer apart from your suggestions, you're seeing some problems with the functionality?

[aeimer]

@danielkesselberg yeah should work, you propably need to merge and try it...

[dboschm]

The blocking change requests are very tiny and additionally a matter of taste. For me the changes are very nice implemented the way they are.

We deceided against your requested changes

We deceided against your requested changes

[kesselb]