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

Allow using strict and compat together #1741 #1955

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

smmhatami
Copy link
Contributor

This is a fix to solve Issue 1741. Resolves #1741. just checked if obj is really an object at the errors occurrence line specified in the issue. I hope it helps.

@jaylinski jaylinski self-assigned this Jun 29, 2023
@jaylinski
Copy link
Member

Note: failing test was not caused by this PR and fixed in #1964. Needs rebase.

Signed-off-by: Seyed Mohammad Mahdi Hatami <hatamik7@gmail.com>
@jaylinski
Copy link
Member

@smmhatami I added a test for your PR, but it seems that combining compat and strict compiler-options causes another issue, because the template in the test doesn't compile correctly. I'm not yet sure why.

@jaylinski jaylinski changed the title handled error for using strict and compat together #1741 Allow using strict and compat together #1741 Aug 31, 2023
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.

Exception when the strict and compat compiler options are used together
2 participants