Skip to content

Remove obsolete ansible directory#1920

Merged
osma merged 1 commit intomainfrom
cleanup-ansible
Feb 2, 2026
Merged

Remove obsolete ansible directory#1920
osma merged 1 commit intomainfrom
cleanup-ansible

Conversation

@osma
Copy link
Member

@osma osma commented Feb 2, 2026

Reasons for creating this PR

The top level directory ansible was created 7 years ago. We have not actually used Ansible in Skosmos development, and the configuration files are no longer valid for current Skosmos. This PR removes the directory along with its contents. We can always revive it if it turns out to be useful later on.

Link to relevant issue(s), if any

  • n/a

Description of the changes in this PR

  • remove ansible directory and all its contents

Known problems or uncertainties in this PR

None

Checklist

  • phpUnit tests pass locally with my changes
  • I have added tests that show that the new code works, or tests are not relevant for this PR (e.g. only HTML/CSS changes)
  • The PR doesn't reduce accessibility of the front-end code (e.g. tab focus, scaling to different resolutions, use of .sr-only class, color contrast)
  • The PR doesn't introduce unintended code changes (e.g. empty lines or useless reindentation)

@osma osma added this to the 3.1 milestone Feb 2, 2026
@osma osma self-assigned this Feb 2, 2026
@osma osma added the maintenance Dependency changes, security updates, infrastructure tweaks & general mainenance label Feb 2, 2026
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 2, 2026

@osma
Copy link
Member Author

osma commented Feb 2, 2026

Waiting for tests to pass, then I will merge this.

@osma osma merged commit f86f7f2 into main Feb 2, 2026
16 checks passed
@github-project-automation github-project-automation bot moved this to Issue/PR closed in Skosmos 3.x Backlog Feb 2, 2026
@osma osma deleted the cleanup-ansible branch February 2, 2026 10:18
@osma osma mentioned this pull request Feb 2, 2026
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Dependency changes, security updates, infrastructure tweaks & general mainenance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant