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

Maintenance 2022-12-07 #107

Merged
merged 24 commits into from
Dec 7, 2022
Merged

Maintenance 2022-12-07 #107

merged 24 commits into from
Dec 7, 2022

Conversation

eclipxe13
Copy link
Owner

This is a maintenance release fo fix the continuous integration workflow and append pending development changes.

  • Fix test CertificadoTest::testConstructWithValidExample() to allow quoted slashes on name.
  • Add phpdoc to the method Certificate::getCertificateName().
    The value can contain quoted slashes \/ depending on the OpenSSL version.
  • Update script tests/validate.php to validate CFDI 3.3 or CFDI 4.0.
  • Add return types to some methods:
    • Status::comparableValue and Status::__toString.
    • Discoverer::discoverInFile.
  • Improve TestCase::installCertificate(): It doesn't depend on the certificate's file name to install correctly.
  • Update GitHub build workflow:
    • Update GH Workflows: Remove deprecated ::set-output & ::save-state.
    • Split full build actions to individual jobs.
    • Split Windows and Linux testing.

Now, it does not depends on the name of the certificate to perform a correct installation
This value is different from on different openssl versions.
3.x: uses quoted slashes
1.1.x: does not use quoted slashes
Matrix is not used on full-build job
- List actions
- Remove whitespace
@eclipxe13 eclipxe13 changed the title Maintenance 2022-12-05 Maintenance 2022-12-07 Dec 7, 2022
@eclipxe13 eclipxe13 merged commit 61c64f1 into master Dec 7, 2022
@eclipxe13 eclipxe13 deleted the development branch May 26, 2023 03:15
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.

None yet

1 participant