Skip to content

Conversation

@llucax
Copy link
Contributor

@llucax llucax commented Sep 8, 2023

The following fixes are pulled in:

  • Replace the section-index plugin with navigation.indexes
  • Fix consumer power formula
  • Add test for consumer power formula

llucax and others added 9 commits September 6, 2023 10:16
Signed-off-by: Leandro Lucarella <[email protected]>
The `section-index` plugin has a bug[^1] that causes the navigation bar
to display a wrong text for the **API Reference** link. There is a fix
for it, but `mkdocs-material` already provides the feature natively[^2],
so we can just use it instead, as it is much less likely to have issues.

[^1]: oprypin/mkdocs-section-index#18
[^2]: https://squidfunk.github.io/mkdocs-material/setup/setting-up-navigation/#section-index-pages

Signed-off-by: Leandro Lucarella <[email protected]>
Signed-off-by: Leandro Lucarella <[email protected]>
The `section-index` plugin has a bug[^1] that causes the navigation bar
to display a wrong text for the **API Reference** link. There is a fix
for it, but `mkdocs-material` already provides the feature natively[^2],
so we can just use it instead, as it is much less likely to have issues.

[^1]: oprypin/mkdocs-section-index#18
[^2]:
https://squidfunk.github.io/mkdocs-material/setup/setting-up-navigation/#section-index-pages
Fixes frequenz-floss#640.
All grid meters have to be summed for this formula but in the current
implemantation there was minus pushed to the operator stack. In this
commit the sign error has been corrected.

Signed-off-by: Matthias Wende <[email protected]>
This adds a tests with two grid meters.

Signed-off-by: Matthias Wende <[email protected]>
Signed-off-by: Matthias Wende <[email protected]>
* tag 'v0.25.1':
  Update release notes
  Add test for consumer power formula
  Fix consumer power formula
  Prepare release notes for the release
  Replace the `section-index` plugin with `navigation.indexes`
  Clear release notes
@llucax llucax requested a review from a team as a code owner September 8, 2023 12:15
@llucax llucax requested a review from Marenz September 8, 2023 12:15
@github-actions github-actions bot added part:tests Affects the unit, integration and performance (benchmarks) tests part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) part:data-pipeline Affects the data pipeline labels Sep 8, 2023
@llucax llucax self-assigned this Sep 8, 2023
@llucax llucax added this to the v1.0.0-rc milestone Sep 8, 2023
@llucax llucax added the type:bug Something isn't working label Sep 8, 2023
@llucax llucax enabled auto-merge September 8, 2023 12:15
@llucax
Copy link
Contributor Author

llucax commented Sep 8, 2023

Enabled auto-merge.

@llucax llucax added the cmd:skip-release-notes It is not necessary to update release notes for this PR label Sep 8, 2023
@llucax llucax added this pull request to the merge queue Sep 8, 2023
Merged via the queue into frequenz-floss:v0.x.x with commit 3b52fa9 Sep 8, 2023
@llucax llucax deleted the merge-v0.25.1 branch September 8, 2023 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cmd:skip-release-notes It is not necessary to update release notes for this PR part:data-pipeline Affects the data pipeline part:tests Affects the unit, integration and performance (benchmarks) tests part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) type:bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants