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

Clean-up module folders #40

Open
3 of 4 tasks
barkz opened this issue Apr 6, 2023 · 2 comments
Open
3 of 4 tasks

Clean-up module folders #40

barkz opened this issue Apr 6, 2023 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@barkz
Copy link
Contributor

barkz commented Apr 6, 2023

If we are not using files as part of the repo because of refactoring they should be removed along with any other files.

  • E.g. Remove exchange.ps1, restorebackup.ps1, exposebackup.ps1 which is not referenced in the folder.

  • Eg. Readme.md files in all directories. Should we be doing this or just have the central readme.md on the main page?

  • E.g. Remove sqlserver-scripts folder if we are not using.
  • Other unused files.
@barkz barkz added the enhancement New feature or request label Apr 6, 2023
@aguzev
Copy link
Collaborator

aguzev commented Apr 7, 2023

SQL Server scripts have their own repository at https://github.com/PureStorage-OpenConnect/sqlserver-scripts.git . Contents of sqlserver-scripts are supposed to go there.

@aguzev aguzev added the needs clarification(s) Issue needs further clarification(s) label Apr 12, 2023
@barkz
Copy link
Contributor Author

barkz commented Apr 12, 2023

Understood. So when we release, we need to push updated sqlserver-scripts to that repo and remove the local folder of sqlserver-scripts from the toolkit3 repo. Make sense?

@aguzev aguzev removed the needs clarification(s) Issue needs further clarification(s) label Apr 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants