Skip to content

initial draft for template args rules#195

Merged
technophile-04 merged 3 commits intobetafrom
templating-args-rules
Feb 14, 2025
Merged

initial draft for template args rules#195
technophile-04 merged 3 commits intobetafrom
templating-args-rules

Conversation

@technophile-04
Copy link
Collaborator

Description:

The initial draft for initial rules.

I think this should probably be merged after #175.

After #175 we can update other config files to follow the same patterns like updating tailwind.config, nextjs.config etc.

Copy link
Member

@rin-st rin-st left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great now!

Copy link
Member

@carletex carletex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Love this!

@technophile-04 technophile-04 changed the base branch from main to beta February 14, 2025 07:12
@technophile-04
Copy link
Collaborator Author

Thanks all! Merging this for now, we can also keep iterating on it if we come across some more cases 🙌

@technophile-04 technophile-04 merged commit b89b22c into beta Feb 14, 2025
1 check passed
@technophile-04 technophile-04 deleted the templating-args-rules branch February 14, 2025 07:16
@rin-st
Copy link
Member

rin-st commented Feb 14, 2025

Since templating was changed from $$$var to $$var$$ #175 (comment), part of this PR is not actual. Fix: #215

@rin-st rin-st mentioned this pull request Feb 14, 2025
@technophile-04 technophile-04 mentioned this pull request Jun 5, 2025
3 tasks
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.

3 participants