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

revision device list #620

Merged
merged 2 commits into from
Aug 6, 2019

Conversation

HomeAutoUser
Copy link
Contributor

  • 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, ...)
  • sort, added some device
  • remove (only receive) comment
  • What is the current behavior? (You can also link to an open issue here)
  • devices not list

- sort, added some device
- remove  (only receive) comment
@HomeAutoUser
Copy link
Contributor Author

:-D

FEHLER von einem Test "nur" weil die Bezeichnung angepasst wurde bzw. ergänzt wurde ;-)
Das Schreit nach Nachbesserung am Test smile

@sidey79
Copy link
Contributor

sidey79 commented Aug 5, 2019

Das ist doch immer noch der Perl 5.18 Fehler :(

@HomeAutoUser
Copy link
Contributor Author

Ich war zuversichtlich das bei so einer Änderung es vielleicht klappt.

@HomeAutoUser
Copy link
Contributor Author

Die jetzigen Fehler kann ich auch nicht deuten.
Deine Perl Anpassung reichte noch nicht aus.

@sidey79
Copy link
Contributor

sidey79 commented Aug 6, 2019

In deinem Branch habe ich nichts angepasst.

Du müsstest ein merge von dev-r34 in deinen Branch machen. Danach sollte es funktionieren

@coveralls
Copy link

Pull Request Test Coverage Report for Build 1653

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at ?%

Totals Coverage Status
Change from base Build 1652: 0.0%
Covered Lines:
Relevant Lines: 0

💛 - Coveralls

2 similar comments
@coveralls
Copy link

Pull Request Test Coverage Report for Build 1653

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at ?%

Totals Coverage Status
Change from base Build 1652: 0.0%
Covered Lines:
Relevant Lines: 0

💛 - Coveralls

@coveralls
Copy link

coveralls commented Aug 6, 2019

Pull Request Test Coverage Report for Build 1653

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at ?%

Totals Coverage Status
Change from base Build 1652: 0.0%
Covered Lines:
Relevant Lines: 0

💛 - Coveralls

@HomeAutoUser
Copy link
Contributor Author

@sidey79,
nein deine Änderungen haben zu keinem Erfolg geführt. der 5.20er Test fällt nun negativ aus :-(

@sidey79
Copy link
Contributor

sidey79 commented Aug 6, 2019

Das ist ein anderer Fehler, da wird das UnitTests Modul nicht gefunden.

@sidey79
Copy link
Contributor

sidey79 commented Aug 6, 2019

open json file SD_Device_ProtocolList was not possible -1

Es scheint mir eher, dass etwas mit der JSON Datei nicht passt. :(

@HomeAutoUser
Copy link
Contributor Author

HomeAutoUser commented Aug 6, 2019

Dann ist es vielleicht ein url Fehler?

Ich habe die aktuelle JSON Datei (Branch dev_JSON_v2) geladen und diese funktioniert.
http://jsoneditoronline.org/?id=bb9513fab8044faa8f49014efc2fd70f

EDIT: diesen Path
my $jsonStr = GetFileFromURL("https://raw.githubusercontent.com/RFD-FHEM/SIGNALduino_TOOL/dev_JSON_v1/FHEM/lib/SD_Device_ProtocolList.json",4,"",1,4);

Bitte auf den Branch dev_JSON_v2 schwenken.
WENN dann alle TESTS mal ohne FEHLER durchlaufen möchte ich gern diese DATEI ins MASTER setzen das wir IMMER den Branch MASTER als Bezug nutzen. Es bringt nichts, wenn ein Zwischenbranch verlinkt is.

@sidey79
Copy link
Contributor

sidey79 commented Aug 6, 2019

Das war ein temporäres Problem, jetzt geht es wieder.

@elektron-bbs
Copy link
Contributor

elektron-bbs commented Aug 6, 2019

Trotz allem wundert mich die Fehlermeldung gleich am Beginn der Ausgabe von Travis:

alter PR:

5.26 is not installed; attempting download
Downloading archive: https://storage.googleapis.com/travis-ci-language-archives/perl/binaries/ubuntu/14.04/x86_64/perl-5.26.tar.bz2
$ sudo tar xjf perl-5.26.tar.bz2 --directory /
git.checkout
1.44s$ git clone https://github.com/RFD-FHEM/RFFHEM.git RFD-FHEM/RFFHEM

neuer PR:

5.26@cover is not installed; attempting download
Downloading archive: https://storage.googleapis.com/travis-ci-language-archives/perl/binaries/ubuntu/16.04/x86_64/perl-5.26@cover.tar.bz2
$ sudo tar xjf perl-5.26@cover.tar.bz2 --directory /
bzip2: (stdin) is not a bzip2 file.
tar: Child returned status 2
tar: Error is not recoverable: exiting now
git.checkout
1.95s$ git clone https://github.com/RFD-FHEM/RFFHEM.git RFD-FHEM/RFFHEM

@sidey79
Copy link
Contributor

sidey79 commented Aug 6, 2019

Der Download der Perl Version mit dem Standard Vorgehen klappt nicht, da ich @cover an die Version gehangen habe.

Das ist aber kein Problem, da der Perl Travis helper die Installation der Perl Version übernimmt.

@HomeAutoUser HomeAutoUser merged commit 95b8b0e into RFD-FHEM:dev-r34 Aug 6, 2019
@HomeAutoUser HomeAutoUser deleted the dev_r34_doc_devices branch August 6, 2019 20:27
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.

4 participants