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

update yaml create script #2378

Merged

Conversation

mizutoki79
Copy link
Contributor

@mizutoki79 mizutoki79 commented Jul 8, 2020

  • Refactor indent and space
  • Correct the cologne leak
  • Change the destination of the WriteAllLines' first argument to an absolute path
  • Run from any directory to be stored in /manifests//
Microsoft Reviewers: Open in CodeFlow

@denelon
Copy link
Contributor

denelon commented Jul 9, 2020

@mizutoki79 it looks like we just made changes to this file due to another issue. Would you mind resolving conflicts?

@mizutoki79
Copy link
Contributor Author

OK..
I'll resolve tonight (JST) or by day after tomorrow.

@ghost ghost removed the Needs: author feedback label Jul 9, 2020
@mizutoki79
Copy link
Contributor Author

@denelon I resolved. Please check.

@denelon
Copy link
Contributor

denelon commented Jul 28, 2020

@mizutoki79 Sorry, I didn't see the mention.

@denelon denelon merged commit 6f4e05f into microsoft:master Jul 28, 2020
megamorf pushed a commit to megamorf/winget-pkgs that referenced this pull request Aug 25, 2020
* this commit removes PowerShell Core only features that were introduced with microsoft#2378 to ensure compatibility with PSv5
* adds comment based help
* use consistently cased Cmdlet and variable names
* improves input data related error handling
* adds PS code regions for automatic code folding
* automates the creation of the manifest file at the correct destination path
* corrects the use of Write-Host and Write-Output

Fixes microsoft#3061
KevinLaMS pushed a commit that referenced this pull request Aug 27, 2020
* Make script PSv5 compatible again, apply some PS best practices

* this commit removes PowerShell Core only features that were introduced with #2378 to ensure compatibility with PSv5
* adds comment based help
* use consistently cased Cmdlet and variable names
* improves input data related error handling
* adds PS code regions for automatic code folding
* automates the creation of the manifest file at the correct destination path
* corrects the use of Write-Host and Write-Output

Fixes #3061

* Adhere to YAML formatting conventions

* Use cross-platform new line handling

Co-authored-by: Sebastian Neumann <sebastian.neumann@experienceone.com>
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.

2 participants