Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

support for individual file to package/zip and loosen directory structure #4

Closed
DumpySquare opened this issue Jun 7, 2020 · 5 comments
Labels
enhancement New feature or request wontfix This will not be worked on
Milestone

Comments

@DumpySquare
Copy link
Member

Need support to specify a single file to be validated and zipped. Currently, the "packageTemplateSet" only accepts a directory. This would be incredibly useful when developing individual templates and while initially getting familiar with the tool.

Furthermore, being able to specify multiple files to be packaged would also be helpful to allow users to have other types of files in the same directory and not be so limited by directory file structure. Or skip files that don't pass validation.

This is on a windows 10 machine with vscode/powershell

PS C:\Users\starlord\f5-ts-testing> npx fast packageTemplateSet .\tempSet\hello.mst
cli.js packageTemplateSet <templateSetPath> [dst]

build a package for a given template set

Positionals:
  templateSetPath  path to the directory containing template sources                                          [required]
  dst              optional location for the built package (defaults to the current working directory)

Options:
  --help     Show help                                                                                         [boolean]
  --version  Show version number                                                                               [boolean]

Error: ENOTDIR invalid cwd C:\Users\starlord\f5-ts-testing\tempSet\hello.mst
    at Glob._readdirError (C:\Users\starlord\f5-ts-testing\node_modules\glob\glob.js:592:21)
    at C:\Users\starlord\f5-ts-testing\node_modules\glob\glob.js:553:12
    at FSReqCallback.oncomplete (fs.js:154:23) {
  path: 'C:\\Users\\starlord\\f5-ts-testing\\tempSet\\hello.mst',
  code: 'ENOTDIR'
}
PS C:\Users\starlord\f5-ts-testing>
@mstokes-f5
Copy link
Contributor

I like the idea of letting packageTemplateSet take a path to either a file or to a directory. I believe the same could be done for validate, which would make validateTemplateSet redundant. To make things consistent, I think we'd then want to rename packageTemplateSet to just package.

One area where this gets tricky is with dependent files. The CLI tool would have to scan the file to determine dependencies (e.g., types defined in a json file, other templates referenced via allOf/oneOf/anyOf, etc.) and make sure they get packaged too.

@DumpySquare
Copy link
Member Author

Some time has passed on this, and knowing more about the template set flow, this feature is less important. I think some examples of how to use this would help. close if you want to focus table this idea for now.

@mstokes-f5 mstokes-f5 changed the title [RFE] support for individual file to package/zip and loosen directory structure support for individual file to package/zip and loosen directory structure Aug 6, 2020
@shyawnkarim
Copy link

We are not tracking this enhancement internally with ID MYSTIQUE-324.

@shyawnkarim shyawnkarim added this to the backlog milestone Feb 8, 2021
@walterk82
Copy link

walterk82 commented Mar 12, 2021

Having this feature would be stellar. For our organization, we write our own template framework for merging iRules into an AS3 declaration at build/deploy time. If this feature existed within FAST then we would not need our homebrew any more as iRules would have a friendly development environment by being included in the resulting AS3 declaration.

Without this feature we could only have limited deployment with FAST as we use a lot of iRules and base64 encoding them within the template is not consumable for end-users.

@shyawnkarim
Copy link

Closing.

After a review of this request, we have decided to not implement this feature.

@shyawnkarim shyawnkarim added the wontfix This will not be worked on label Dec 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

4 participants