Skip to content

Master: Fix tests on Windows, add Ruby 3.3, and cleanup CI#687

Merged
pitbulk merged 1 commit intoSAML-Toolkits:masterfrom
johnnyshields:master-fix-tests-windows
Jul 9, 2024
Merged

Master: Fix tests on Windows, add Ruby 3.3, and cleanup CI#687
pitbulk merged 1 commit intoSAML-Toolkits:masterfrom
johnnyshields:master-fix-tests-windows

Conversation

@johnnyshields
Copy link
Copy Markdown
Collaborator

On Windows, the DateTime class struggles with Float values for certain methods like #next_year some reason.

@johnnyshields johnnyshields force-pushed the master-fix-tests-windows branch 2 times, most recently from 99ec80d to 27d6c02 Compare July 8, 2024 12:48
@johnnyshields
Copy link
Copy Markdown
Collaborator Author

johnnyshields commented Jul 8, 2024

@pitbulk this is ready to be merged.

The JRuby Zlib::BufError failures are transient and can be ignored.

@johnnyshields johnnyshields force-pushed the master-fix-tests-windows branch from 27d6c02 to c21cb22 Compare July 8, 2024 12:55
@johnnyshields johnnyshields changed the title Master: Fix tests on Windows and cleanup CI Master: Fix tests on Windows, add Ruby 3.3, and cleanup CI Jul 8, 2024
@johnnyshields johnnyshields force-pushed the master-fix-tests-windows branch from c21cb22 to 1b74fab Compare July 8, 2024 13:03
@johnnyshields
Copy link
Copy Markdown
Collaborator Author

@pitbulk this can be merged independently of the other PRs.

@pitbulk
Copy link
Copy Markdown
Collaborator

pitbulk commented Jul 8, 2024

This PR has a mix of the code to be sent to V2 and the one to master branch

See https://github.com/SAML-Toolkits/ruby-saml/pull/687/files#diff-d57c932f52c1cf52ca2f9028672cc7d038bf9c8200484ddc80e63a212a68e6b3

@johnnyshields johnnyshields force-pushed the master-fix-tests-windows branch from 1b74fab to 0f8da70 Compare July 9, 2024 03:22
@johnnyshields
Copy link
Copy Markdown
Collaborator Author

@pitbulk oops! Sorry about that, its now fixed and ready to merge.

@pitbulk pitbulk merged commit 6e33ed3 into SAML-Toolkits:master Jul 9, 2024
@pitbulk
Copy link
Copy Markdown
Collaborator

pitbulk commented Jul 9, 2024

@johnnyshields, We should also apply this on V2

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