Codecov Report

Merging #245 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #245   +/-   ##
=======================================
  Coverage   74.59%   74.59%           
=======================================
  Files          20       20           
  Lines         681      681           
=======================================
  Hits          508      508           
  Misses        128      128           
  Partials       45       45

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d89e42d...7980b8f. Read the comment docs.

@bmoffatt Did you have any thoughts on this? Thanks.

@bmoffatt Did you have any thoughts on this? Thanks.

I'd rather not do a semver-major change. I'm ok with adding and alias type CodePipelineEvent = CodePipelineJobEvent to help with the ambiguity.

@bmoffatt I understand the reluctance for a semver-major but this will trip a lot of people up in the future even with the alias, it breaks the principle of least surprise all the other Events are what they say they are. This is what I did originally - Assumed that CodePipelineEvent was the same as CodeBuildEvent and then when digging in found that CodePipelineEvent was something completely unexpected!

We won't be merging intentionally breaking changes to the event types at this time. I've tagged #244 as requiring a v2 release to keep track of the request.

For #247, a different type name can be used.