Skip to content

Revert b89550f51a7e996779192e126a3d487f4e0a4bcb (re-enable Windows encoding test) #146

Closed
dduportal wants to merge 6 commits intojenkinsci:masterfrom
dduportal:helpdesk-4568
Closed

Revert b89550f51a7e996779192e126a3d487f4e0a4bcb (re-enable Windows encoding test) #146
dduportal wants to merge 6 commits intojenkinsci:masterfrom
dduportal:helpdesk-4568

Conversation

@dduportal
Copy link
Copy Markdown

@dduportal dduportal commented Mar 4, 2025

Ref. jenkins-infra/helpdesk#4568 (comment)

This change allows the Jenkins Infra team to retry Windows builds and test the encoding settings.
It is a manual revert of b89550f which is hidden inside #145

It must stay in draft until jenkins-infra/helpdesk#4568 is updated.

This change allows the Jenkins Infra team to retry Windows builds and test the encoding settings.

Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
Save a little time evaluating changes
@MarkEWaite
Copy link
Copy Markdown
Contributor

I reduced the number of tests to run on this branch in order to reduce the cycle time. No need to run more tests than necessary.

Commit d7adf37

I left it running on both Windows and Linux because I recall that there were some specific conditions that need at least one Linux build.

@jglick
Copy link
Copy Markdown
Member

jglick commented Mar 10, 2025

#145 (comment) FWIW

@MarkEWaite
Copy link
Copy Markdown
Contributor

Will fix the test, since it should not be charset dependent

@MarkEWaite MarkEWaite closed this Mar 10, 2025
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.

3 participants