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

fixs for jcenter down. #248

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

fixs for jcenter down. #248

wants to merge 3 commits into from

Conversation

icehong
Copy link
Contributor

@icehong icehong commented Mar 1, 2025

PullRequest

Great that you are thinking about submitting a PullRequest. Please keep them small if not otherwise possible. Contact me before you are creating a big PR to avoid unnecessary work and rebasing of big PRs. Also try to add tests - I am not religious about that but prefer PRs backed by tests. This project has everything setup for Espresso UI and Unit-tests. Also the existing unit and UI-tests muss pass before submitting a PullRequest. Please base the PullRequests on dev.

PullRequest

Great that you are thinking about submitting a PullRequest!

Please keep them small if not otherwise possible. Contact me before you are c
reating a big PR to avoid unnecessary work and rebasing of big PRs. Also try to
add tests - I am not dogmatic about that but prefer PRs backed by tests. This
project has everything setup for Espresso UI and Unit-tests. Also the existing
unit and UI-tests muss pass before submitting a PullRequest.

Please base the PullRequests on the branch named dev

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.

1 participant