Skip to content

fix(igniter): supports both code and strings as config values#166

Merged
grzuy merged 9 commits intomainfrom
igniter-more
Sep 23, 2025
Merged

fix(igniter): supports both code and strings as config values#166
grzuy merged 9 commits intomainfrom
igniter-more

Conversation

@grzuy
Copy link
Copy Markdown
Collaborator

@grzuy grzuy commented Sep 19, 2025

follow up #165

@grzuy grzuy changed the title refactor(igniter): support both code and strings as config values refactor(igniter): supports both code and strings as config values Sep 19, 2025
@grzuy grzuy marked this pull request as ready for review September 22, 2025 14:00
@grzuy grzuy changed the title refactor(igniter): supports both code and strings as config values feat(igniter): supports both code and strings as config values Sep 22, 2025
@grzuy
Copy link
Copy Markdown
Collaborator Author

grzuy commented Sep 22, 2025

Related to the test failures: ash-project/igniter#332.

@grzuy grzuy changed the title feat(igniter): supports both code and strings as config values fix(igniter): supports both code and strings as config values Sep 23, 2025
@grzuy grzuy merged commit a10c670 into main Sep 23, 2025
10 checks passed
@grzuy grzuy deleted the igniter-more branch September 23, 2025 13:41
@grzuy
Copy link
Copy Markdown
Collaborator Author

grzuy commented Sep 23, 2025

Related to the test failures: ash-project/igniter#332.

Went back to support only runtime root level config and not inside conditional if block for now.
To do that we need to figure out some failures when calling igniter functions.

Will do separate future PRs.

@grzuy
Copy link
Copy Markdown
Collaborator Author

grzuy commented Sep 23, 2025

Related to the test failures: ash-project/igniter#332.

Went back to support only runtime root level config and not inside conditional if block for now. To do that we need to figure out some failures when calling igniter functions.

Will do separate future PRs.

#168

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