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

added phpstan setup #813

Open
wants to merge 15 commits into
base: release-candidate
Choose a base branch
from
Open

added phpstan setup #813

wants to merge 15 commits into from

Conversation

akshayurankar48
Copy link

Description

Screenshots

Types of changes

How has this been tested?

Checklist:

  • My code is tested
  • My code passes the PHPCS tests
  • My code follows accessibility standards
  • My code has proper inline documentation
  • I've included any necessary tests
  • I've included developer documentation
  • I've added proper labels to this pull request

sushmak02 and others added 13 commits March 18, 2024 13:39
@sushmak02
Merge pull request #782 from brainstormforce/release-candidate
3d0584a 
Release Candidate to Master
@sushmak02
Merge pull request #793 from brainstormforce/release-candidate
dcce8a3 
Build: Version 1.6.27
@sushmak02
Merge pull request #796 from brainstormforce/release-candidate
6eedcac 
Build version 1.6.28
@sushmak02
Merge pull request #803 from brainstormforce/release-candidate
5997f88 
Build: v1.6.29
@sushmak02
Merge pull request #807 from brainstormforce/release-candidate
d8616b3 
Version 1.6.30
@sushmak02
Fix: Cauing an error due to file include
92137ea
@sushmak02
Added file conditions
2579555
@sushmak02
Added a class exists check
6e0c960
@vijetaR
Tag and changelog
5e27fbe
@vijetaR
Updated changelog
632dcf3
@sushmak02
Merge pull request #811 from brainstormforce/file-include-bug
80d2303 
Fix: Causing an error due to file include
@vijetaR
Tags limit
6df0d9c
added phpstan setup
9557d2b
vijetaR
vijetaR reviewed May 15, 2024
View reviewed changes
Copy link
Contributor

@vijetaR vijetaR left a comment

Choose a reason for hiding this comment

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

I hope RC is up-to-date.

vijetaR
vijetaR reviewed May 23, 2024
View reviewed changes
Copy link
Contributor

@vijetaR vijetaR left a comment •
edited

Choose a reason for hiding this comment

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

@akshayurankar48 Please update the PR. Resolve conflicts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vijetaR vijetaR vijetaR approved these changes

@sushmak02 sushmak02 sushmak02 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@akshayurankar48 @themuddfamily @sushmak02 @vijetaR