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

Clean up flexmeasures show formatting #540

Merged
merged 3 commits into from Nov 24, 2022
Merged

Clean up flexmeasures show formatting #540

merged 3 commits into from Nov 24, 2022

Conversation

Flix6x
Copy link
Contributor

@Flix6x Flix6x commented Nov 24, 2022

I noticed a few inconsistencies in our formatting.

@Flix6x
Clean up table formatting
74a2dc8 
Signed-off-by: F.N. Claessen <felix@seita.nl>
@Flix6x
Streamline how the term ID is printed out
cdc4241 
Signed-off-by: F.N. Claessen <felix@seita.nl>
@Flix6x Flix6x requested a review from nhoening November 24, 2022 22:21
@Flix6x Flix6x self-assigned this Nov 24, 2022
@coveralls
Copy link
Collaborator

coveralls commented Nov 24, 2022 •
edited

Pull Request Test Coverage Report for Build 3543993298

  • 0 of 9 (0.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.02%) to 65.002%
Changes Missing Coverage Covered Lines Changed/Added Lines %
flexmeasures/cli/data_show.py 0 9 0.0%
Totals Coverage Status
Change from base Build 3518850719: 0.02%
Covered Lines: 6586
Relevant Lines: 9513
💛 - Coveralls

@Flix6x
Changelog entry
1d6b86b 
Signed-off-by: F.N. Claessen <felix@seita.nl>
@Flix6x Flix6x added this to the 0.12.0 milestone Nov 24, 2022
@Flix6x Flix6x merged commit f54cea2 into main Nov 24, 2022
@Flix6x Flix6x deleted the clean-up-flexmeasures-show-formatting branch November 24, 2022 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nhoening nhoening nhoening approved these changes

Assignees

@Flix6x Flix6x

Labels
CLI
Projects
None yet
Milestone
0.12.0
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@Flix6x @coveralls @nhoening