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

chore: misc updates and improvements made while you were gone, take 2 #583

Merged
merged 8 commits into from
Jun 6, 2024

Conversation

sverrehu
Copy link
Contributor

@sverrehu sverrehu commented Feb 19, 2024

This is a "While you were gone, take 2".

  • Updates certain GitHub actions to get rid of a lot of warnings.
  • Updates most dependencies to the most recent version.
  • When pretty-printing JSON, split arrays in one entry per line. This makes it much easier to spot differences in .cluster-state.
  • Display the warning for disabled remote state verification just once.
  • Make it possible to disable warnings printed when projects contain no topics.
  • Make it possible to disable warnings printed when streams have no writers.

This PR is one commit behind master, since the most recently merged contribution breaks the unit tests. (Maybe revert it to have unit tests work again.)

* chore(deps): update most dependencies
@purbon purbon self-requested a review February 19, 2024 14:44
@sverrehu
Copy link
Contributor Author

Hi, @purbon! As noted in the description, the tests will continue to fail for this PR (and others) as long as the previous merge to master is not reverted or fixed by another merge.

@sverrehu sverrehu changed the title chore(deps): update most dependencies chore: misc updates and improvements made while you were gone Mar 10, 2024
@sverrehu sverrehu changed the title chore: misc updates and improvements made while you were gone chore: misc updates and improvements made while you were gone, take 2 Mar 10, 2024
@purbon purbon merged commit e75d005 into kafka-ops:master Jun 6, 2024
2 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