Skip to content

Added tests, psalm and phpcs fixes#12

Merged
alexmerlin merged 4 commits into3.0from
issue-10
Aug 18, 2023
Merged

Added tests, psalm and phpcs fixes#12
alexmerlin merged 4 commits into3.0from
issue-10

Conversation

@Jurj-Bogdan
Copy link
Copy Markdown
Member

#10

@Jurj-Bogdan
Copy link
Copy Markdown
Member Author

@arhimede a symfony insight badge would be needed

@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 17, 2023

Codecov Report

❗ No coverage uploaded for pull request base (3.0@e984ec2). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             3.0       #12   +/-   ##
=======================================
  Coverage       ?   100.00%           
  Complexity     ?        49           
=======================================
  Files          ?         7           
  Lines          ?       132           
  Branches       ?         0           
=======================================
  Hits           ?       132           
  Misses         ?         0           
  Partials       ?         0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Copy Markdown
Member

@alexmerlin alexmerlin left a comment

Choose a reason for hiding this comment

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

Please restore file permissions (755 -> 644) for the following files:

  • .gitignore
  • CHANGELOG.md
  • LICENSE.md
  • OSSMETADATA
  • README.md
  • composer.json

Comment thread README.md Outdated
Comment thread composer.json Outdated
Comment thread test/FlashMessengerTest.php Outdated
Comment thread test/FlashMessengerTest.php Outdated
Copy link
Copy Markdown
Member

@alexmerlin alexmerlin left a comment

Choose a reason for hiding this comment

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

I just noticed that the permissions have changed on the files in the src directory too.
Please restore those as well (755 -> 644).

@alexmerlin alexmerlin linked an issue Aug 18, 2023 that may be closed by this pull request
added SymfonyInsight badge
@alexmerlin alexmerlin merged commit 698e9da into 3.0 Aug 18, 2023
@alexmerlin alexmerlin deleted the issue-10 branch August 18, 2023 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

green badge Bump psr/http-message to: ^1.0 || ^2.0 codecov badge Code quality

2 participants