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.

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

Add session type to Window Manager entry #142

Closed
wants to merge 22 commits into from

Conversation

Tobilike
Copy link
Contributor

@Tobilike Tobilike commented Jan 7, 2024

I add a check for the Display-Server-Protokoll
session

Description

The system checks whether a Display Manager is available; if so, it is displayed under Window Manager. A system variable is checked, if this cannot be found, nothing happens.

Reason and / or context

It is useful because there are 2 of them and it is not clear which is used.

How has this been tested ?

On 2 Systems

Types of changes :

  • Bug fix (non-breaking change which fixes an issue)
  • Typo / style fix (non-breaking change which improves readability)
  • [?] New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist :

  • [IF NEEDED] I have updated the README.md file accordingly ;
  • [IF NEEDED] I have updated the test cases (which pass) accordingly ;
  • [IF BREAKING] This pull request targets next Archey version branch ;
  • My changes looks good ;
  • I agree that my code may be modified in the future ;
  • My code follows the code style of this project (PEP8).

@ingrinder ingrinder added the enhancement ⬆️ Implements a new feature, fixes or improves existing ones label Jan 7, 2024
@ingrinder ingrinder changed the title Session Add session type to Window Manager entry Jan 7, 2024
@ingrinder
Copy link
Collaborator

Hi again!

As in #141, do you think you would be able to rebase this onto master with just the relevant commits? If not, we can do it for you, though note it would require a force push which will clobber your local copy of the branch (particularly any changes not pushed).

@HorlogeSkynet
Copy link
Owner

Closing here in favor of #145.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ⬆️ Implements a new feature, fixes or improves existing ones
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants