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.

Sign up for GitHub

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

Jump to bottom

Clear marketplace cache during upgrade #227

Merged
merged 2 commits into from
Jan 14, 2024
Merged

Clear marketplace cache during upgrade #227

merged 2 commits into from
Jan 14, 2024

Conversation

J-N-K
Copy link
Member

@J-N-K J-N-K commented Jan 5, 2024

@J-N-K
Clear marketplace cache during upgrade
42d7eb1 
Signed-off-by: Jan N. Klug <github@klug.nrw>
BClark09
BClark09 requested changes Jan 7, 2024
View reviewed changes
Copy link
Member

@BClark09 BClark09 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @J-N-K! With the suggested change should we wait until your PR in core is merged?

resources/control-runtime/preinst Outdated Show resolved Hide resolved
@J-N-K @BClark09
Update resources/control-runtime/preinst
8200a01 
Co-authored-by: Ben Clark <ben@benjyc.uk>
Signed-off-by: J-N-K <github@klug.nrw>
@J-N-K
Copy link
Member Author

J-N-K commented Jan 7, 2024

Correct, this will otherwise create issues.

BClark09
BClark09 approved these changes Jan 7, 2024
View reviewed changes
Copy link
Member

@BClark09 BClark09 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, I'll watch the other PR and merge when appropriate. Thanks again!

@BClark09 BClark09 self-assigned this Jan 7, 2024
@J-N-K
Copy link
Member Author

J-N-K commented Jan 14, 2024

@BClark09 You can merge now

@BClark09 BClark09 merged commit 10061ac into openhab:main Jan 14, 2024
1 check passed
@J-N-K J-N-K deleted the marketplace branch January 14, 2024 22:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BClark09 BClark09 BClark09 approved these changes

Assignees

@BClark09 BClark09

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@J-N-K @BClark09