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

Update dependencies #69

Merged
merged 2 commits into from
Apr 29, 2024

Conversation

denysvitali
Copy link
Contributor

@denysvitali denysvitali commented Apr 5, 2024

This PR updates the dependencies and is part of the split of #68.
A lot of code style changes here are due to the auto-formatter of Flutter (why isn't the project using it?)

This PR depends on #71, review only 73624b2 until #71 is merged

@denysvitali denysvitali mentioned this pull request Apr 5, 2024
@kolaente
Copy link
Member

kolaente commented Apr 5, 2024

Can you rebase this on latest main?

@Benimautner
Copy link
Collaborator

Re: why the project wasn't using flutter format: Basically, we just never started haha. It's definitely a good idea. As @kolaente said, please rebase all your PRs on latest main so the changelog doesn't include all the formatting.

And I don't want to push the minsdk all the way up to Android 13, so if there's any way you can downgrade the dependencies, that would be amazing!

@Benimautner Benimautner merged commit 1d538d6 into go-vikunja:main Apr 29, 2024
2 checks passed
@denysvitali denysvitali deleted the feature/bump-dependencies branch April 29, 2024 12:04
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

3 participants