Skip to content

Conversation

@gaelicWizard
Copy link
Contributor

Description

Change the functions in theme/powerline-naked/powerline.base and theme/powerline/powerline.base to match more closely. Alsö, lint theme/powerline-naked.

Motivation and Context

By updating them to match more closely, we may be able to eliminate the duplicate file in the future. It may be possible to just extend the main file to handle the use case of the alternative implementations.

How Has This Been Tested?

Since I haven't used the powerline* themes much, this is hard for me to test as I'm not used to what "normal" looks like.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code follows the code style of this project.
  • If my change requires a change to the documentation, I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • If I have added a new file, I also added it to clean_files.txt and formatted it using lint_clean_files.sh.
  • I have added tests to cover my changes, and all the new and existing tests pass.

@gaelicWizard gaelicWizard force-pushed the theme/powerline-naked branch 5 times, most recently from b7e768f to 940f571 Compare January 24, 2022 20:17
@gaelicWizard gaelicWizard force-pushed the theme/powerline-naked branch 2 times, most recently from aabf1bb to 3925421 Compare January 26, 2022 19:44
@gaelicWizard gaelicWizard force-pushed the theme/powerline-naked branch 5 times, most recently from 9c1cf2b to 002b6a7 Compare February 13, 2022 06:04
@gaelicWizard gaelicWizard changed the title theme/powerline-naked: harmonize powerline.base files DRAFT: theme/powerline-naked: harmonize powerline.base files Feb 20, 2022
@seefood seefood marked this pull request as ready for review April 30, 2025 11:43
@seefood seefood self-assigned this Apr 30, 2025
@seefood seefood merged commit c68eca0 into Bash-it:master Apr 30, 2025
5 of 6 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