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

Release 1.0.0-beta9 #101

Merged
merged 14 commits into from
Jan 24, 2025
Merged

Release 1.0.0-beta9 #101

merged 14 commits into from
Jan 24, 2025

Conversation

andybroomfield
Copy link
Contributor

@andybroomfield andybroomfield commented Jan 14, 2025

* adds feedback form

* coding standards fixes

* adds a README.md

* removes dump()

* adds status message after installing module

* updates README
@andybroomfield
Copy link
Contributor Author

Bunch of failed tests, but should that hold up release as this may be updates to PHPstan / PHPcs?

@finnlewis
Copy link
Member

Good question, @millnut or @ekes can you tell with a quick glance whether the PHP coding standards and static analysis check failures are a) introduced by changes to phpstan / phpcs and b) should we fix them before release and if so c) are they quick to fix?

@finnlewis finnlewis requested review from ekes and millnut January 21, 2025 10:43
@millnut
Copy link
Member

millnut commented Jan 21, 2025

@finnlewis phpstan errors are real as they won't be included in the project baseline as forms is not part of the main project.

The coding standards should be fixed. The phpstan error I can add them to the baseline however we don't have any other case where a module that is not included in the main project is added to the baseline, so unsure if we want to start doing this.

@finnlewis
Copy link
Member

@andybroomfield is happy to release with errors, but we might want to fix them if we can.

If we can rattle through and address the phstan issues, that would be great!

@finnlewis
Copy link
Member

If we can't fix the phpstan issues, let's create a separate issue for that and get this release out!

@millnut
Copy link
Member

millnut commented Jan 23, 2025

@finnlewis already fixed the phpstan errors today :) Just with Adnan to take a look #105 due to the PHPUnit failures. But we haven't had a successful PHPUnit test since a couple of months ago

@andybroomfield
Copy link
Contributor Author

@Adnan-cds Ok for this to be released?

Copy link
Contributor

@Adnan-cds Adnan-cds left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@Adnan-cds
Copy link
Contributor

Ok for this to be released?

Sure.

@andybroomfield andybroomfield merged commit 9092a56 into release-1.x Jan 24, 2025
10 of 11 checks passed
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.

5 participants