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

Analyzer #14

Merged
merged 5 commits into from
Apr 6, 2022
Merged

Analyzer #14

merged 5 commits into from
Apr 6, 2022

Conversation

Abdullah104
Copy link
Contributor

No description provided.

@pr-Mais
Copy link
Owner

pr-Mais commented Feb 15, 2022

Hey @Abdullah104 , thanks for the contribution!
Could you please fix the conflicts?

@Abdullah104
Copy link
Contributor Author

I will work on it as soon as possible

@Abdullah104
Copy link
Contributor Author

I fixed the conflicts. However, I just wanted to note that two of the files that had the conflict were committed with compilation time errors. Specifically, the SharedPreferencesService didn't have the type-defined value getters and setters, but I didn't change them because I didn't know if I should. If you want me to fix it. I will do it.

@pr-Mais
Copy link
Owner

pr-Mais commented Feb 23, 2022

Would you fix it please

@Abdullah104
Copy link
Contributor Author

Yes sure

@Abdullah104
Copy link
Contributor Author

This may seem weird, but I just opened the app to fix the error, and the error just wasn't there anymore. I think it may have been a problem with my editor. So sorry for the unnecessary delay.

@pr-Mais
Copy link
Owner

pr-Mais commented Apr 6, 2022

Thanks!

@pr-Mais pr-Mais merged commit d1ab443 into pr-Mais:main Apr 6, 2022
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.

None yet

2 participants