Skip to content

Improve service syntax related docs and tests nits#5991

Merged
lafriks merged 8 commits intowoodpecker-ci:mainfrom
6543-forks:improve-nits
Jan 19, 2026
Merged

Improve service syntax related docs and tests nits#5991
lafriks merged 8 commits intowoodpecker-ci:mainfrom
6543-forks:improve-nits

Conversation

@6543
Copy link
Member

@6543 6543 commented Jan 19, 2026

collection of refactorings and improvements from #5988 but unrelated to it's change

  • docs: fix environment syntax in services
  • test: extend test for services schema
  • test: make TestLintErrors more resilient
  • refactor: move types.ContainerList into own file
  • test: extend ReSerialize test from

@6543 6543 changed the title Improve nits Improve config service syntax related docs and tests nits Jan 19, 2026
@6543 6543 added documentation docu & docs refactor delete or replace old code tests related to tests or other things CI check before merge labels Jan 19, 2026
@6543 6543 requested a review from qwerty287 January 19, 2026 12:30
@woodpecker-bot
Copy link
Contributor

woodpecker-bot commented Jan 19, 2026

Surge PR preview deployment was removed

@qwerty287
Copy link
Contributor

qwerty287 commented Jan 19, 2026

How is this related to the config services?

@6543 6543 changed the title Improve config service syntax related docs and tests nits Improve service syntax related docs and tests nits Jan 19, 2026
@6543
Copy link
Member Author

6543 commented Jan 19, 2026

well title is bad ... its about the syntax in the woodpecker config that relates to service but not the service stuff itself just code,tetsts,docs around it

it does not change how it works just add more test coverage and fix some docu nits

@6543 6543 requested a review from qwerty287 January 19, 2026 16:38
@lafriks lafriks merged commit d1b7e35 into woodpecker-ci:main Jan 19, 2026
7 checks passed
@woodpecker-bot woodpecker-bot mentioned this pull request Jan 19, 2026
1 task
@6543 6543 deleted the improve-nits branch January 20, 2026 01:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation docu & docs refactor delete or replace old code tests related to tests or other things CI check before merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants