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

Replace favicon #267

Merged
merged 7 commits into from
Sep 11, 2024
Merged

Replace favicon #267

merged 7 commits into from
Sep 11, 2024

Conversation

mmaksimovic
Copy link
Collaborator

Description

Instead of using Wasp's logo as a favicon, use a free bee icon template.

Copy link
Collaborator

@sodic sodic left a comment

Choose a reason for hiding this comment

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

Hey, it seems you accidentally committed Mac OS's .DS_Store directories.

That directory should probably be listed in the top-level .gitignore file.
It's strange that this doesn't happen more often since more than half the team is on a Mac.

@infomiho Correct me if I'm missing something and you Mac guys deal with these directories differently.

If not, @mmaksimovic please add .DS_Store to .gitignore and remove them from this PR (enough to delete those folders, git add ., git commit, git push).

@mmaksimovic
Copy link
Collaborator Author

@sodic Fixed!

Looks like we have some flaky tests, the last commit I made was empty, and the tests passed 😅

@sodic
Copy link
Collaborator

sodic commented Sep 11, 2024

the last commit I made was empty, and the tests passed

No worries, that's perfectly fine. If tests are passing before an empty commit, they should be passing on the empty commit too (since it didn't change anything).

What's strange is that one of the tests failed in the first place, but it seems to have been a network issue. I'll look into it right away.

Btw, there's still one .DS_Store left :)

@mmaksimovic
Copy link
Collaborator Author

ahhh, my bad, done!

@@ -1,2 +1,3 @@
/template/app/migrations
/template/app/package-lock.json
.DS_Store
Copy link
Collaborator

Choose a reason for hiding this comment

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

Looks good 👍 we have the same in the Wasp compiler .gitignore

@mmaksimovic mmaksimovic merged commit cff34f2 into main Sep 11, 2024
1 check passed
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.

3 participants