Ignoring fields with struct tag expr:"-" and make "in" return false for unexported fields - #806

Merged

antonmedv merged 6 commits into

expr-lang:masterfrom

diegommm:allow-ignoring-struct-member-with-struct-tag

Jun 30, 2025

diegommm commented Jun 24, 2025•
edited
Loading

Contributor

Fixes#805 and #807

allow ignoring struct member using a - in expr struct tag

401009f

diegommm mentioned this pull request

Feature request: allow ignoring struct fields with struct tag expr:"-" #805

Closed

diegommm commented

View reviewed changes


		fmt.Printf("%v", output)

		// Output : Hello World

diegommmJun 24, 2025

ContributorAuthor

This is unrelated but I realized that this example test was not being run because there is a space between Output and :, so I removed the space and the example test is now running and is successful.

diegommm commented

View reviewed changes

Comment on lines -532 to +597

		// Output: Hello, you, world!
		// Output: Hello, you, world!

diegommmJun 24, 2025

ContributorAuthor

Same as above: this is unrelated but I realized that this example test was not being run because there is a space between Output and :, so I removed the space and the example test is now running and is successful.

fix in operator

6bd3eb5

antonmedv reviewed

View reviewed changes

builtin/lib.go Outdated

antonmedv reviewed

View reviewed changes

checker/nature/utils.go Outdated

antonmedv reviewed

View reviewed changes

checker/nature/utils.go Outdated

antonmedv reviewed

View reviewed changes

checker/nature/utils.go Outdated

antonmedv reviewed

View reviewed changes

vm/runtime/runtime.go Outdated

simplify logic

56d0528

diegommm mentioned this pull request

Operator "in" can see unexported struct fields #807

Closed

diegommm added 2 commits

June 27, 2025 17:05

add regression test forexpr-lang#807

eecf371

fixesexpr-lang#807: make in operator return false for unexported str…

1089aa9

…uct fields

diegommm changed the title ~~Allow ignoring struct fields with struct tag expr:"-"~~Ignoring fields with struct tag expr:"-" and make "in" return false for unexported fields

diegommm requested a review from antonmedv

June 27, 2025 20:23

Merge branch 'master' into allow-ignoring-struct-member-with-struct-tag

27a4dbf

antonmedv approved these changes

View reviewed changes

antonmedv merged commit 85d0be8 into expr-lang:master

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment