Skip to content

Restructure output directories to output/#25

Merged
marcarl merged 3 commits into
mainfrom
feature/restructure-output-dirs
Jan 4, 2026
Merged

Restructure output directories to output/#25
marcarl merged 3 commits into
mainfrom
feature/restructure-output-dirs

Conversation

@marcarl
Copy link
Copy Markdown
Collaborator

@marcarl marcarl commented Jan 4, 2026

Sammanfattning

Samlar all genererad output under output/ med tydlig struktur för bättre organisation och för att undvika oavsiktliga commits av genererade filer.

Ny katalogstruktur

sfs-processor/
├── sfs_docs/          # Input: Nedladdade JSON
├── output/
│   ├── md/            # Output: Markdown (tidigare SFS/)
│   ├── html/          # Output: HTML
│   └── git/           # Output: Git export
└── ...

Ändringar

Konfiguration

  • .gitignore: Rensat bort oanvänd sfs-output/, output/ täcker allt
  • pyproject.toml: Lagt till sfs_docs/* och output/* i coverage omit

Dokumentation

  • README.md: Uppdaterat alla exempel från --output SFS--output output/md

GitHub Workflows (4 filer)

  • fetch-sfs-workflow.yml: Ändrat output och git add-kommando
  • html-export-workflow.yml: Ändrat output och AWS S3 sync-sökväg
  • testdocs-workflow.yml: Ändrat output-katalog
  • upcoming-changes-workflow.yml: Uppdaterat sökvägar för temporal scripts

Fördelar

  • Tydligare struktur: Separation mellan input (sfs_docs/) och output (output/)
  • Enklare städning: rm -rf output/ rensar all genererad data
  • Säkrare: output/ redan ignorerad i .gitignore → undviker oavsiktliga commits
  • Konsekvent: All output samlas under samma root-katalog

Risker & Breaking Changes

⚠️ Breaking change: Användare som förväntar sig SFS/ katalogen måste uppdatera sina scripts/kommandon.

Test plan

  • Kör python sfs_processor.py --input sfs_json --output output/md --formats md-markers
  • Verifiera att filer hamnar i output/md/
  • Testa HTML-export till output/html/
  • Verifiera att workflows fungerar (kan testas manuellt med workflow_dispatch)
  • Kontrollera att coverage-rapporter exkluderar output-kataloger

🤖 Generated with Claude Code

marcarl and others added 2 commits January 4, 2026 21:26
Samlar all genererad output under output/ med tydlig struktur:
- output/md/ - Markdown-filer (tidigare SFS/)
- output/html/ - HTML-filer

## Ändringar

### Konfiguration
- .gitignore: Ta bort oanvänd sfs-output/, output/ täcker allt
- pyproject.toml: Lägg till sfs_docs/* och output/* i coverage omit

### Dokumentation
- README.md: Uppdatera exempel från --output SFS → --output output/md

### GitHub Workflows (4 filer)
- fetch-sfs-workflow.yml: SFS → output/md, git add ändrat
- html-export-workflow.yml: SFS → output/html, aws s3 sync uppdaterat
- testdocs-workflow.yml: SFS → output/md
- upcoming-changes-workflow.yml: SFS/ → output/md/ i temporal scripts

## Fördelar
- Tydligare separation mellan input (sfs_docs/) och output (output/)
- All genererad data under samma root-katalog
- Enklare att rensa: rm -rf output/
- output/ redan ignorerad i .gitignore - undviker oavsiktliga commits

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
data/ ska innehålla källdata och konfiguration, inte genererad output.
kommande.yaml genereras från markdown-filer och är därför output.

Ändringar:
- temporal/upcoming_changes.py: UPCOMING_CHANGES_FILE_PATH → output/kommande.yaml
- .gitignore: Ta bort data/kommande.yaml (täcks av output/)
- upcoming-changes-workflow.yml: git add output/kommande.yaml
@marcarl marcarl changed the title Omstrukturera output-kataloger till output/ Restructure output directories to output/ Jan 4, 2026
Add improvements from feature/improved-pyproject:
- Enable branch coverage (branch = true)
- Add abstractmethod to exclude_lines for better coverage reporting
- Keep output/* omit (covers output/md, output/html, etc.)
@marcarl marcarl merged commit 400e559 into main Jan 4, 2026
3 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.

1 participant