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

Action not accepting multiple sql files or folder (despite being documented) #28

Closed
abhinaba-chakraborty-by opened this issue Jan 20, 2021 · 4 comments
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@abhinaba-chakraborty-by

As per the documentation here:

  sql-file:
    description: 'Path to SQL script file. *.sql or a folder to deploy'

So we should be able to pass a directory name or pass wildcard like *.sql. But when done so, it fails complaining:
(for wildcard)

Error: Muliple files found matching pattern ./abc/xyz/*.sql

and , for folder,

Error: Invalid sql file path provided as input ./abc/xyz/

Is this feature not supported yet? If so, can the documentation please be corrected?

@abhinaba-chakraborty-by abhinaba-chakraborty-by added the need-to-triage Requires investigation label Jan 20, 2021
@aksm-ms aksm-ms self-assigned this Jan 21, 2021
@aksm-ms aksm-ms added documentation Improvements or additions to documentation and removed need-to-triage Requires investigation labels Jan 21, 2021
@aksm-ms
Copy link
Contributor

aksm-ms commented Jan 21, 2021

@abhinaba-chakraborty-by Currently the action does not support multiple files. I have updated the documentation.

@aksm-ms aksm-ms closed this as completed Jan 21, 2021
@abhinaba-chakraborty-by
Copy link
Author

@aksm-ms Ok. Thanks. And also is this action not supported in Linux environment ?? Coz I got the error "This action is not supported on a non-windows environment."

@aksm-ms
Copy link
Contributor

aksm-ms commented Jan 21, 2021

@abhinaba-chakraborty-by Currently linux is not supported. There is already an enhancement request tracking this #11

@joe-schmirler
Copy link

@aksm-ms Multiple files still cannot be ran, despite the readme still has an example of multiple scripts: See 'Deploy SQL scripts to an Azure Database with a temporary firewall rule'.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

3 participants