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

fix: update twitter to X #328 #331

Merged
merged 5 commits into from
Apr 22, 2024
Merged

Conversation

jeevikasirwani
Copy link
Contributor

changed the logo and name

changed the logo and name
@MandyMeindersma
Copy link
Contributor

Thank you @jeevikasirwani

We have been playing around with our linting (sorry to change in in the middle of your pr)

Do you mind re linting and then I'll merge?

@MandyMeindersma
Copy link
Contributor

oh or npm run lint:fix should fix it :)

@jeevikasirwani
Copy link
Contributor Author

oh or npm run lint:fix should fix it :)

IN MY FORK BRANCH RIGHT?

@MandyMeindersma
Copy link
Contributor

absolutely :)

@MandyMeindersma
Copy link
Contributor

Hey @jeevikasirwani I would still love to have your contribution so if you want to run the linting command that would be great, other wise I will merge this into another branch and then commit the linting errors then merge that (that way we still have your commit and work recorded!) 😄

@MandyMeindersma MandyMeindersma merged commit 6ffac4b into devedmonton:main Apr 22, 2024
5 checks passed
@MandyMeindersma
Copy link
Contributor

@arashsheyda saved us again with the auto linting bot! Thank you

@allcontributors add @jeevikasirwani for code

Copy link
Contributor

@MandyMeindersma

I've put up a pull request to add @jeevikasirwani! 🎉

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.

2 participants