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.

Sign up for GitHub

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

Jump to bottom

Nutgram 4 #358

Merged
merged 222 commits into from
May 30, 2023
Merged

Nutgram 4 #358

merged 222 commits into from
May 30, 2023

Conversation

sergix44
Copy link
Member

@sergix44 sergix44 commented Mar 1, 2023 •
edited by Lukasss93
Loading

Task List

Final tasks

@sergix44 sergix44 added the wip Work in progress label Mar 1, 2023
@sergix44 sergix44 requested a review from Lukasss93 as a code owner March 1, 2023 16:27
@sergix44 sergix44 marked this pull request as draft March 1, 2023 16:27
sergix44 and others added 17 commits March 1, 2023 17:49
@sergix44
drop all the laravel related stuff
8b5c3a7
@sergix44
Merge pull request #359 from nutgram/new/drop_laravel_stuff
f739891
@Lukasss93
Rename namespace
e622fdc
@Lukasss93
First attempt to convert ChatMemberStatus to enum
529aed7
@sergix44
patch typefaker
2bcaaee
@sergix44
move config to a dedicated class
2925882
@sergix44 @StyleCIBot
Apply fixes from StyleCI
08dde23 
[ci skip] [skip ci]
@sergix44
Merge pull request #364 from nutgram/analysis-a6NMZ7
8f11f4d
@Lukasss93
Remove tests matrix
75378bc
@Lukasss93
Remove only
286b554
@Lukasss93
Convert some attributes to enum
48840f4
@Lukasss93
Remove getType method from ChatMember class
abcf642
@Lukasss93
Revert "Remove tests matrix"
4c2e447 
This reverts commit 75378bc.
@Lukasss93
Convert UpdateTypes attribute to enum
e76d84f
@Lukasss93
Convert MessageTypes attribute to enum
cfcee72
@Lukasss93 @StyleCIBot
Apply fixes from StyleCI
a59e821 
[ci skip] [skip ci]
@Lukasss93
Merge pull request #367 from nutgram/analysis-01nVwm
4335d72 
Apply fixes from StyleCI
@Lukasss93
Merge remote-tracking branch 'origin/master' into 4.x
5973d3b 
# Conflicts:
#	src/Handlers/Handler.php
codeclimate[bot]
codeclimate bot reviewed May 13, 2023
View reviewed changes
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 15399 lines exceeds the maximum allowed for the inline comments feature.

codeclimate[bot]

This comment was marked as outdated.

Sign in to view

codeclimate[bot]

This comment was marked as outdated.

Sign in to view

@sergix44 sergix44 marked this pull request as ready for review May 16, 2023 16:50
codeclimate[bot]
codeclimate bot reviewed May 22, 2023
View reviewed changes
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 15423 lines exceeds the maximum allowed for the inline comments feature.

codeclimate[bot]
codeclimate bot reviewed May 23, 2023
View reviewed changes
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 15423 lines exceeds the maximum allowed for the inline comments feature.

codeclimate[bot]
codeclimate bot reviewed May 23, 2023
View reviewed changes
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 15423 lines exceeds the maximum allowed for the inline comments feature.

codeclimate[bot]
codeclimate bot reviewed May 23, 2023
View reviewed changes
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 15421 lines exceeds the maximum allowed for the inline comments feature.

codeclimate[bot]
codeclimate bot reviewed May 25, 2023
View reviewed changes
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 15444 lines exceeds the maximum allowed for the inline comments feature.

codeclimate[bot]
codeclimate bot reviewed May 29, 2023
View reviewed changes
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 15466 lines exceeds the maximum allowed for the inline comments feature.

codeclimate[bot]
codeclimate bot reviewed May 30, 2023
View reviewed changes
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 15501 lines exceeds the maximum allowed for the inline comments feature.

@codeclimate
Copy link

codeclimate bot commented May 30, 2023

Code Climate has analyzed commit 2068468 and detected 4 issues on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 4

The test coverage on the diff in this pull request is 98.8% (60% is the threshold).

This pull request will bring the total coverage in the repository to 95.9% (0.2% change).

View more on Code Climate.

@Lukasss93 Lukasss93 merged commit 6d6bf35 into master May 30, 2023
10 checks passed
@Lukasss93 Lukasss93 deleted the 4.x branch May 30, 2023 20:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@codeclimate codeclimate[bot] codeclimate left review comments

@Lukasss93 Lukasss93 Awaiting requested review from Lukasss93 Lukasss93 is a code owner

Assignees
No one assigned
Labels
wip Work in progress
Projects
Nutgram 4.0
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@sergix44 @Lukasss93