Page MenuHomePhorge
Differential D249

Add rules parser
ClosedPublic
Actions

Authored by tusooa on Dec 4 2025, 11:51 AM.
Tags
None
Referenced Files
F85803207: D249.1791453441.diff
Wed, Oct 7, 2:57 AM
F85803201: D249.1791453289.diff
Wed, Oct 7, 2:54 AM
F85803198: D249.1791453267.diff
Wed, Oct 7, 2:54 AM
F85803194: D249.1791453106.diff
Wed, Oct 7, 2:51 AM
F85802490: D249.1791409927.diff
Tue, Oct 6, 2:52 PM
Subscribers
None

Details

Reviewers
None
Group Reviewers
O2: lilybuild
Commits
rBe8e281fc3a4f: Add rules parser
Summary

Add a parser to parse and evaluate rules:if expressions. It does not support regex yet.

Type: add

Test Plan

Verify unit tests pass.

Diff Detail

Repository
rB lilybuild
Branch
tusooa/stacked
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 477
Build 937: Invoke BuildbotBuildbot build #430
Build 936: arc lint + arc unit

Unit TestsFailed
View All

TimeTest
0 msCoverage (unit-test-master)

https://b.lily-is.land/#/builders/11/builds/31

0 msCoverage (unit-test-worker)

https://b.lily-is.land/#/builders/12/builds/31

Event Timeline

tusooa created this revision.Dec 4 2025, 11:51 AM
Owners added a reviewer: O2: lilybuild.Dec 4 2025, 11:51 AM
Harbormaster returned this revision to the author for changes because remote builds failed.Dec 4 2025, 11:53 AM
Harbormaster failed remote builds in B477: Diff 744!
tusooa updated this revision to Diff 745.Dec 4 2025, 11:55 AM

Remove + (no-backtrack) modifier

Harbormaster completed remote builds in B478: Diff 745.Dec 4 2025, 11:58 AM
tusooa requested review of this revision.Dec 4 2025, 11:58 AM
This revision was not accepted when it landed; it landed in state Needs Review.Dec 5 2025, 9:58 AM
Closed by commit rBe8e281fc3a4f: Add rules parser. · Explain Why
This revision was automatically updated to reflect the committed changes.
tusooa added a commit: rBe8e281fc3a4f: Add rules parser.

Revision Contents
Changeset List

PathSizeCoverage (All)Coverage (Touched)Packages
lilybuild/
lilybuild/
ci_syntax/
120 lines2%
Loading...
O2: lilybuild
tests/
ci_syntax/
118 lines--O2: lilybuild

Diff 744

View Options

lilybuild/lilybuild/ci_syntax/rules.py

Loading...
View Options

lilybuild/lilybuild/tests/ci_syntax/rules_test.py

Loading...