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

Add seasons infrastructure and basic display #9648

Closed
wants to merge 316 commits into from

Conversation

venix12
Copy link
Member

@venix12 venix12 commented Dec 16, 2022

(a big) part of #8736

image

@venix12
Copy link
Member Author

venix12 commented Dec 16, 2022

am i doing something wrong or is lint just having a stroke 🤨

@notbakaneko
Copy link
Collaborator

It means the new code added more warnings; we set a max limit on warnings for existing code, new code should not be adding lint errors or new warnings.

If running yarn lint returns the same number of warnings, you may need to clear the lint cache.

@venix12
Copy link
Member Author

venix12 commented Dec 17, 2022

Well, no matter what I tried, I got 158 on both this branch and master, tried clearing cache too. 🤷

EDIT: Apparently had a wrong node version installed in my environment for some reason 😅, changing to the right one fixed the issue.

@venix12
Copy link
Member Author

venix12 commented Jan 10, 2023

remember to never use git rebase irresponsibly!

replaced by #9725 as it's probably way easier than fixing this one =D

@venix12 venix12 closed this Jan 10, 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.

5 participants