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

release 3.4.4 #864

Merged
merged 41 commits into from
Jul 5, 2020
Merged

release 3.4.4 #864

merged 41 commits into from
Jul 5, 2020

Conversation

sidey79
Copy link
Contributor

@sidey79 sidey79 commented Jul 4, 2020

  • Please check if the PR fulfills these requirements
  • Tests for the changes have been added / modified (needed for for bug fixes / features)
  • commandref has been added / updated (needed for bug fixes / features)
  • CHANGED has been updated (needed for bug fixes / features)
  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

  • 00_SIGNALduino.pm
    Fix some perlcritic 5 warnings
    some operator fixes Wrong operator for eq 00_SIGNALduino #835
    Changed stringrefs to coderefs

10_FS10
revised for perlcritic

10_SD_GT
revised for perlcritic (#847)

  • 14_SD_UT.pm
    add attribute UTfrequency in (add attribute UTfrequency in 14_SD_UT.pm #845)
    new attribute UTfrequency for individual transmission frequency
    Multiple query "$ attrName eq 'model'" removed
    change commandref
    replace "foreach my $keys (keys %models)" with "if ( exists $models{$attrValue}" in line 1756
    replace "foreach my $keys (sort keys %{$models{RC_10}{buttons}})" with "if ( exists $models{RC_10}{buttons}{$button} )" in line 1845
    replace "foreach my $keys (keys %models)" with "grep { $models{$_}{hex_lengh} =~ /$hex_lengh/ } keys %models;" in line 1745
    delete "$attr{$name}{room} = "SD_UT" if( not defined( $attr{$name}{room} ) );"
    change search for models with same hex_length
    Reference to autocreate added
    rename hex_lengh in hex_length
    Check values attribute UTfrequency added
    Documentation for UTfrequency attribute added

  • SD_ProtocolData.pm

new protocol 104 for remote control TR60C-1 (#844)
new protocol 99 NAVARIS touch light switch (#841)
new protocol 98 for wireless door bell GEA-028DB (#839)

  • What is the current behavior? (You can also link to an open issue here)

  • What is the new behavior (if this is a feature change)?

  • Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

no

  • Other information:

sidey79 and others added 30 commits April 13, 2020 22:59
Changed stringrefs to coderefs
Cange string sub references to realcoderefs
some operator fixes #835
Fixed some operators
Fixed some operators
Fixed some operators
fixes operator #836
Fix compare against string
tests revised to Test2
* SD_UT new attribute UTfrequency for individual transmission frequency
Multiple query "$ attrName eq 'model'" removed
change commandref
replace "foreach my $keys (keys %models)" with "if ( exists $models{$attrValue}" in line 1756
replace "foreach my $keys (sort keys %{$models{RC_10}{buttons}})" with "if ( exists $models{RC_10}{buttons}{$button} )" in line 1845
replace "foreach my $keys (keys %models)" with "grep { $models{$_}{hex_lengh} =~ /$hex_lengh/ } keys %models;" in line 1745
delete "$attr{$name}{room}	= "SD_UT"	if( not defined( $attr{$name}{room} ) );"
change search for models with same hex_length
@sidey79 sidey79 added this to the 3.4.3 milestone Jul 4, 2020
@sidey79 sidey79 changed the title Dev r34 release 3.4.4 Jul 4, 2020
@sidey79 sidey79 marked this pull request as ready for review July 4, 2020 23:11
@sidey79
Copy link
Contributor Author

sidey79 commented Jul 4, 2020

@HomeAutoUser
@elektron-bbs

So damit, der Sprung für 3.5 nicht so groß wird, habe ich die 3.4.4 vorbereitet.

@coveralls
Copy link

coveralls commented Jul 4, 2020

Pull Request Test Coverage Report for Build 3520

  • 109 of 584 (18.66%) changed or added relevant lines in 8 files are covered.
  • 24 unchanged lines in 4 files lost coverage.
  • Overall coverage decreased (-0.8%) to 53.464%

Changes Missing Coverage Covered Lines Changed/Added Lines %
FHEM/90_SIGNALduino_un.pm 0 1 0.0%
FHEM/14_SD_WS09.pm 1 3 33.33%
FHEM/14_SD_WS.pm 0 3 0.0%
FHEM/14_SD_BELL.pm 2 12 16.67%
FHEM/00_SIGNALduino.pm 64 81 79.01%
FHEM/10_SD_GT.pm 11 121 9.09%
FHEM/14_SD_UT.pm 18 139 12.95%
FHEM/10_FS10.pm 13 224 5.8%
Files with Coverage Reduction New Missed Lines %
FHEM/10_FS10.pm 1 6.78%
FHEM/14_SD_UT.pm 2 53.94%
FHEM/10_SD_GT.pm 4 5.91%
FHEM/00_SIGNALduino.pm 17 76.85%
Totals Coverage Status
Change from base Build 2971: -0.8%
Covered Lines: 3311
Relevant Lines: 6193

💛 - Coveralls

@elektron-bbs
Copy link
Contributor

OK

@sidey79 sidey79 merged commit 3925f07 into master Jul 5, 2020
@sidey79 sidey79 deleted the rel-3.4.4 branch July 5, 2020 16:14
@HomeAutoUser
Copy link
Contributor

HomeAutoUser commented Jul 15, 2020

Hallo @sidey79,
die Frage passt hier glaube ich am besten hin.

Die die Modul Version 3.4.4 https://github.com/RFD-FHEM/RFFHEM/releases als Release markiert ist, so können wir doch die dev-r3.5 als Pre-Release einstufen wo wir die kommenden Entwicklungen ergänzen. Oder wie hast du dir das gedacht?

Parallel wäre es günstig, die Firmware Version 3.4.0-dev+20200711 als Release zu markieren (https://github.com/RFD-FHEM/SIGNALDuino/releases) um mit 3.4 zu 3.4 gleich zu bleiben.
Dort würden wir die dev-r3.5 beginnen und sind somit wieder auf einer Ebene.

Achso, die 3.4 könntest du dann in den Master schieben.

@elektron-bbs, entspricht dies auch deinen Vorstellungen?

@sidey79
Copy link
Contributor Author

sidey79 commented Jul 15, 2020

Die Releases beziehen sich immer auf einen commit und stellen quasi einen Schnappschuss dar.
Wenn wir jetzt ein prerelease 3.5.0 anlegen, auf welchen Commit soll es heute und in Zukunft zeigen?

Bei der Firmware ist es eigentlich egal ob diese die gleiche Versionsnummer wie das Modul hat.
Ich habe jetzt verstanden, dass uns keine Probleme mit der 3.4.0 bekannt sind und wollen diese als final veröffentlichen.

@HomeAutoUser
Copy link
Contributor

Wenn wir jetzt ein prerelease 3.5.0 anlegen, auf welchen Commit soll es heute und in Zukunft zeigen?

Das würde meiner Meinung erst Sinn machen, auf diesen #866 Commit.
Bezogen auf die Firmware, würde es nur Sinn machen auf den Commit, welcher folgt, wo das Empfangen von xFSK eingebunden ist. (Diese Firmware lassen wir ja schon laufen und den Commit würde ich vollziehen, wenn der ModulCommit druch ist)

Ich habe jetzt verstanden, dass uns keine Probleme mit der 3.4.0 bekannt sind und wollen diese als final veröffentlichen.

Nein, bisher sind keine Probleme bekannt. Im Gegenteil, wir haben ja mit diem RFD-FHEM/SIGNALDuino@72c9e25 einen Fix behoben. Somit könntest du den Stand RFD-FHEM/SIGNALDuino@74fa948 als "Cut" übernehmen.
Okay, Probleme sind bisher nur die nicht angetasteten wie hier #782 (comment). ( Um nicht einfach alles unter den Tisch zu kehren ;-) )

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.

Wrong operator for eq 00_SIGNALduino
5 participants