Skip to content

chore: Update generator to create 'data/catalogs' by default#136

Merged
eddie-knight merged 5 commits intoprivateerproj:mainfrom
eddie-knight:chore/generator
Oct 13, 2025
Merged

chore: Update generator to create 'data/catalogs' by default#136
eddie-knight merged 5 commits intoprivateerproj:mainfrom
eddie-knight:chore/generator

Conversation

@eddie-knight
Copy link
Contributor

@eddie-knight eddie-knight commented Oct 12, 2025

Also, fixed a bug in the filepaths for non-template files that are copied over.

Also, ignore evaluation_results. We eventually need to fix the fact that "evaluation logs" are being created when not evaluating.

corresponds to privateerproj/plugin-generator-templates#19

Signed-off-by: Eddie Knight <knight@linux.com>
@eddie-knight eddie-knight requested a review from a team as a code owner October 12, 2025 16:04
Copilot AI review requested due to automatic review settings October 12, 2025 16:04
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Updates the generator to place generated catalogs under data/catalogs by default and removes the simplifiedName helper from templates and code.

  • Switch writeCatalogFile output directory to data/catalogs
  • Remove simplifiedName helper function and its template FuncMap entry

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@eddie-knight eddie-knight marked this pull request as draft October 12, 2025 16:15
Signed-off-by: Eddie Knight <knight@linux.com>
@eddie-knight eddie-knight marked this pull request as ready for review October 12, 2025 16:19
@eddie-knight eddie-knight requested a review from jmeridth October 12, 2025 16:19
Signed-off-by: Eddie Knight <knight@linux.com>
@eddie-knight eddie-knight marked this pull request as draft October 12, 2025 16:42
Signed-off-by: Eddie Knight <knight@linux.com>
@eddie-knight eddie-knight marked this pull request as ready for review October 12, 2025 16:58
@eddie-knight eddie-knight merged commit 966035a into privateerproj:main Oct 13, 2025
7 checks passed
@eddie-knight eddie-knight deleted the chore/generator branch October 13, 2025 01:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants