Skip to content

Use is_extedned_id instead of extended_id in Message constructor#455

Merged
felixdivo merged 2 commits intodevelopfrom
fix-is_extended_id
Oct 8, 2018
Merged

Use is_extedned_id instead of extended_id in Message constructor#455
felixdivo merged 2 commits intodevelopfrom
fix-is_extended_id

Conversation

@felixdivo
Copy link
Copy Markdown
Collaborator

@felixdivo felixdivo commented Oct 7, 2018

This needs to be done before printing a warning when extended_id is used.

Part of #424.

@felixdivo felixdivo added this to the 3.1 Release milestone Oct 7, 2018
@codecov
Copy link
Copy Markdown

codecov bot commented Oct 8, 2018

Codecov Report

Merging #455 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #455   +/-   ##
========================================
  Coverage    63.72%   63.72%           
========================================
  Files           55       55           
  Lines         4672     4672           
========================================
  Hits          2977     2977           
  Misses        1695     1695

@felixdivo felixdivo self-assigned this Oct 8, 2018
@felixdivo felixdivo changed the title Use is_expedned_id instead of extended_id in Message constructor Use is_extedned_id instead of extended_id in Message constructor Oct 8, 2018
@felixdivo felixdivo merged commit ea1d2f0 into develop Oct 8, 2018
@felixdivo felixdivo deleted the fix-is_extended_id branch October 8, 2018 10:47
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