- 
                Notifications
    You must be signed in to change notification settings 
- Fork 20
Merge v0.25.1 #652
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
          
     Merged
      
      
    
                
     Merged
            
            Merge v0.25.1 #652
Conversation
  
    
      This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
      Learn more about bidirectional Unicode characters
    
  
  
    
    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
| Enabled auto-merge. | 
              
                    daniel-zullo-frequenz
  
              
              approved these changes
              
                  
                    Sep 8, 2023 
                  
              
              
            
            
  
    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 
  Add this suggestion to a batch that can be applied as a single commit.
  This suggestion is invalid because no changes were made to the code.
  Suggestions cannot be applied while the pull request is closed.
  Suggestions cannot be applied while viewing a subset of changes.
  Only one suggestion per line can be applied in a batch.
  Add this suggestion to a batch that can be applied as a single commit.
  Applying suggestions on deleted lines is not supported.
  You must change the existing code in this line in order to create a valid suggestion.
  Outdated suggestions cannot be applied.
  This suggestion has been applied or marked resolved.
  Suggestions cannot be applied from pending reviews.
  Suggestions cannot be applied on multi-line comments.
  Suggestions cannot be applied while the pull request is queued to merge.
  Suggestion cannot be applied right now. Please check back later.
  
    
  
    
The following fixes are pulled in:
section-indexplugin withnavigation.indexes