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.

Sign up for GitHub

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

Jump to bottom

Make Multi-line Field Exclude ending Newline #77620

Merged
2 commits merged into from
Jan 31, 2023
Merged

Make Multi-line Field Exclude ending Newline #77620

2 commits merged into from
Jan 31, 2023

Conversation

Trenly
Copy link
Contributor

@Trenly Trenly commented Sep 2, 2022 •
edited by ghost
Loading

null

Microsoft Reviewers: Open in CodeFlow

@wingetbot
Copy link
Collaborator

Service Badge  Service Badge  

@wingetbot
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@wingetbot wingetbot added the Validation-Installation-Error Error installing application label Sep 2, 2022
@ghost ghost added the Moderator-Approved One of the Moderators has reviewed and approved this PR label Sep 2, 2022
@ghost
Copy link

ghost commented Sep 2, 2022

Hello @Trenly,

The package manager bot determined there was an issue with installing the application correctly. Please check the application installs correctly. Once repaired, please push an update to your pull request.

Template: msftbot/validationError/installation/general

@ghost ghost added the Needs-Author-Feedback This needs a response from the author. label Sep 2, 2022
@Trenly
Copy link
Contributor Author

Trenly commented Sep 2, 2022

@denelon - Blocking Issue

2022-09-02 04:42:57.503 [CLI ] Caught std::exception: No mapping for the Unicode character exists in the target multi-byte code page.

microsoft/winget-cli#2057

@ghost ghost added Needs-Attention This work item needs to be reviewed by a member of the core team. and removed Needs-Author-Feedback This needs a response from the author. labels Sep 2, 2022
@denelon denelon added Blocking-Issue Manifest validation is blocked by a known issue. and removed Needs-Attention This work item needs to be reviewed by a member of the core team. labels Sep 2, 2022
@wingetbot
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

There was an error handling pipeline event 2eca6e60-9459-438c-849e-6d634acad8ef.

@ghost ghost removed Validation-Installation-Error Error installing application Blocking-Issue Manifest validation is blocked by a known issue. Moderator-Approved One of the Moderators has reviewed and approved this PR labels Jan 31, 2023
@wingetbot
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@wingetbot wingetbot added the Internal-Error An unidentified error occurred in the validation pipeline label Jan 31, 2023
@ghost
Copy link

ghost commented Jan 31, 2023

Hello @Trenly,

The pull request encountered an internal error and has been assigned to a developer to investigate.

Template: msftbot/validationError/internalError

@ghost ghost added the Needs-Attention This work item needs to be reviewed by a member of the core team. label Jan 31, 2023
@ghost ghost assigned stephengillie and zachcarp Jan 31, 2023
@wingetbot
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ghost ghost removed Internal-Error An unidentified error occurred in the validation pipeline Needs-Attention This work item needs to be reviewed by a member of the core team. labels Jan 31, 2023
@wingetbot wingetbot added Azure-Pipeline-Passed Validation pipeline passed. There may still be manual validation requirements. Validation-Completed Validation passed labels Jan 31, 2023
@ghost
Copy link

ghost commented Jan 31, 2023

Hello @wingetbot!

Because this pull request has the Validation-Completed label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost
Copy link

ghost commented Jan 31, 2023

Trenly,

The check-in policies require a moderator to approve PRs from the community.

Our moderators are community volunteers, please be patient and allow them sufficient time to review your submission.

Template: msftbot/requiresApproval/moderator

@ghost ghost added the Moderator-Approved One of the Moderators has reviewed and approved this PR label Jan 31, 2023
@ghost
Copy link

ghost commented Jan 31, 2023

Hello Trenly,
Validation has completed.

Template: msftbot/validationCompleted

@ghost ghost merged commit 8f1d5ea into microsoft:master Jan 31, 2023
@Trenly Trenly deleted the da8cc3cdaef146b29e5ce084cd3206a8 branch January 31, 2023 14:18
@wingetbot
Copy link
Collaborator

Publish pipeline succeeded for this Pull Request. Once you refresh your index, this change should be present.

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ImJoakim ImJoakim ImJoakim approved these changes

@jedieaston jedieaston jedieaston approved these changes

@ItzLevvie ItzLevvie ItzLevvie approved these changes

Assignees
No one assigned
Labels
Azure-Pipeline-Passed Validation pipeline passed. There may still be manual validation requirements. Moderator-Approved One of the Moderators has reviewed and approved this PR Publish-Pipeline-Succeeded Validation-Completed Validation passed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@Trenly @wingetbot @ItzLevvie @jedieaston @ImJoakim @zachcarp @denelon @stephengillie