Bash-it / bash-it

A community Bash framework.
MIT License
14.23k stars 2.29k forks source link

Add new theme - Inretio #2247

Open gytisrepecka opened 6 months ago

gytisrepecka commented 6 months ago

Add new theme called Inretio, along with documentation describing it.

Description

Inretio theme uses inspiration from existing themes: metal and bobby, but adds more details about virtual environment - when Python is available in it, prompt will display that particular Python binary version.

Motivation and Context

Displaying specific Python version is very useful for developers who keep multiple venvs for running their Python code on different runner versions, e.g.: 3.12.x and 3.9.x on the same machine.

How Has This Been Tested?

Tested on usual filesystem folder, Git-tracked folder, in venv without Python and in venv with Python.

Screenshots (if appropriate):

Added in documentation as required: https://github.com/Bash-it/bash-it/pull/2246

Types of changes

Checklist:

gytisrepecka commented 6 months ago

It looks like there is an error in linter:

Bash include file themes/inretio/inretio.theme.bash has bad/missing shellcheck header

Because following is indeed the recommended shebang to properly find BASH location:

#!/usr/bin/env bash

Edit: not related to shebang - actually theme file needed following on top of it:

# shellcheck shell=bash
# shellcheck disable=SC2034 # Expected behavior for themes.
gytisrepecka commented 6 months ago

Currently have no idea what Linter wants me to fix. Is there a usable documentation to understand and troubleshoot Linter issues?

Current documentation hardly helps 🧐