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

Upgrade LinkTap to 0.3.0 #2615

Merged
merged 21 commits into from
Oct 24, 2023
Merged

Upgrade LinkTap to 0.3.0 #2615

merged 21 commits into from
Oct 24, 2023

Conversation

Smart-Gang
Copy link
Contributor

Support for new device types and ioBroker dependency (admin) update

@mcm1957
Copy link
Collaborator

mcm1957 commented Sep 10, 2023

2.0.3 created 9.9.2023

@mcm1957
Copy link
Collaborator

mcm1957 commented Sep 10, 2023

just a feedback for next releases:

Added support for new devices sounds like new functionality. Thus increasing the minor version would have been correct.

The rules for versioning are:

  • increase major if Breaking changes are introduced. This may include new features and bugfixes.
  • increase minor if a new feature was added without breaking things. This may include bugfixes.
  • increase patch if only a bug was fixed without adding new functionality.

@mcm1957
Copy link
Collaborator

mcm1957 commented Sep 10, 2023

@Smart-Gang

Your adapter https://github.com/Smart-Gang/ioBroker.linktap does not use standard iobroker testing.

Please see requirements https://github.com/ioBroker/ioBroker.repositories#requirements-for-adapter-to-get-added-to-the-latest-repository

Adapter needs to have at least Adapter basic testing (installing, running) using GitHub actions. More information in Forum from apollon77 (Just take from other adapters the samples)

You may copy the test-and-release.yml workflow from the example adaptzer or newly created adapter. If you need help, please let uas know.

Please add a comment when you have reviewed and fixed the suggestionsor at least commented the suggestions and you think the adapter is ready for a re-review

@mcm1957 mcm1957 added must be fixed The Adapter request got review/automatic feedback that is required to be fixed before another review and removed STABLE - brand new labels Sep 10, 2023
@mcm1957 mcm1957 added the ON HOLD PR is set ON HOLD due to pending question or major issues. label Sep 26, 2023
@mcm1957
Copy link
Collaborator

mcm1957 commented Oct 17, 2023

I've created a PR to aid in updateing / adding standard tests. (Smart-Gang/ioBroker.linktap#10)
Please merge and check that tests pass.

After merging and checking, please drop a comment to let us continue release management.

NOTE: If there is no reaction within 14 days I will close this PR.

@mcm1957 mcm1957 added question Something needs to be done or answered by the adapter developer and removed ON HOLD PR is set ON HOLD due to pending question or major issues. labels Oct 17, 2023
@Smart-Gang
Copy link
Contributor Author

Integrated Adapter basic tests using Github Actions
Applied new minor version due to compatibility for new devices.

@github-actions github-actions bot added the *📬 a new comment has been added label Oct 18, 2023
@mcm1957 mcm1957 changed the title Upgrade LinkTap to 0.2.3 Upgrade LinkTap to 0.3.0 Oct 18, 2023
@mcm1957
Copy link
Collaborator

mcm1957 commented Oct 19, 2023

Thanks for your work.
Github test feedback still shows a failure. This is not caused by the standard github tests - they are ok - but by appvayour integration:

image

If you do not use appveyor I suggest to remove it from your test configuration. I was told that this canbe achived by

  • open settings poage of github repository
  • go to webhooks
  • remove appveyor webhook

Of course its ok to fix the appveyor setup and to use it if you prefer. I'm sorry but I cannot aid with this testing environment. The goal should be to get a green checkmark as all tests pass.

mcm1957

P.S: appveyor issue is NOT blocking as I can check which tests fail manually - but it should be fixed in future to give you a quick feedback for durtehr work at the adapter. THANK YOU once again for your effort.

@mcm1957 mcm1957 added RE-REVIEW pending (by mcm1957) Changes requested by review have been applied, re-review could be done. and removed question Something needs to be done or answered by the adapter developer must be fixed The Adapter request got review/automatic feedback that is required to be fixed before another review *📬 a new comment has been added labels Oct 19, 2023
@mcm1957
Copy link
Collaborator

mcm1957 commented Oct 19, 2023

0.3.0 created 18.10.2023
manual adapter check OK

wait for minimum time at latest

@mcm1957 mcm1957 added STABLE - brand new and removed RE-REVIEW pending (by mcm1957) Changes requested by review have been applied, re-review could be done. labels Oct 19, 2023
@Smart-Gang
Copy link
Contributor Author

Thanks for your work. Github test feedback still shows a failure. This is not caused by the standard github tests - they are ok - but by appvayour integration:

image

If you do not use appveyor I suggest to remove it from your test configuration. I was told that this canbe achived by

  • open settings poage of github repository
  • go to webhooks
  • remove appveyor webhook

Of course its ok to fix the appveyor setup and to use it if you prefer. I'm sorry but I cannot aid with this testing environment. The goal should be to get a green checkmark as all tests pass.

mcm1957

P.S: appveyor issue is NOT blocking as I can check which tests fail manually - but it should be fixed in future to give you a quick feedback for durtehr work at the adapter. THANK YOU once again for your effort.

Thanks for the info, I stopped using the appveyor and have now removed the webhook as well.

Now all tests should be green.

Thanks for the support.

@github-actions github-actions bot added the *📬 a new comment has been added label Oct 19, 2023
@mcm1957 mcm1957 removed the *📬 a new comment has been added label Oct 20, 2023
@mcm1957
Copy link
Collaborator

mcm1957 commented Oct 20, 2023

0.3.0 created 18.10.2023
18 user
OK

@mcm1957 mcm1957 merged commit 545dd90 into ioBroker:master Oct 24, 2023
22 checks passed
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