Skip to content

Fix the problem with empty entries for apt_packages#71

Merged
h-chmeruk merged 1 commit into
mainfrom
fix/apt_packages-in-readthedocs
May 5, 2026
Merged

Fix the problem with empty entries for apt_packages#71
h-chmeruk merged 1 commit into
mainfrom
fix/apt_packages-in-readthedocs

Conversation

@h-chmeruk
Copy link
Copy Markdown
Contributor

@h-chmeruk h-chmeruk commented Apr 29, 2026

Which issue(s) are closed by this pull request?

During the recent update of the pyfar package in PR#927 using the template, I noticed that a new entry had been created under apt_packages in the .readthedocs.yml file, even though there was only one package specified in the copier variable apt_packages
image

Changes proposed in this pull request:

  • Expand the implementation so that entries are not created if they consist only of spaces
  • Adjust the tests

@h-chmeruk h-chmeruk added this to the v0.1.0 milestone Apr 29, 2026
@h-chmeruk h-chmeruk added the bug Something isn't working label Apr 29, 2026
@h-chmeruk h-chmeruk requested a review from ahms5 April 29, 2026 08:16
@h-chmeruk h-chmeruk changed the title Fixed the problem with empty entries for apt_packages, adjusted tests Fix the problem with empty entries for apt_packages Apr 29, 2026
@h-chmeruk h-chmeruk merged commit 1f522bf into main May 5, 2026
15 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in Weekly Planning May 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants