Skip to content

Conversation

MGaetan89
Copy link
Member

Both deckard repositories (for Gradle and Maven) haven't been updated in years, and have been archived last February.
This commit removes the references to these projects, as they don't really help getting started with Robolectric.

Here's the updated section of the readme:
Screenshot 2025-06-26 at 14 39 03

Both deckard repositories (for Gradle and Maven) haven't been updated in years, and have been archived last February.
This commit removes the references to these projects, as they don't really help getting started with Robolectric.
@MGaetan89 MGaetan89 self-assigned this Jun 26, 2025
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

The pull request removes references to the deckard repositories from the README.md file and updates the installation instructions with current dependencies. The changes improve the clarity and relevance of the README for new users.

@Test
public void clickingButton_shouldChangeMessage() {
try (ActivityController<MyActivity> controller = Robolectric.buildActivity(MyActivity.class)) {
controller.setup(); // Moves Activity to RESUMED state
controller.setup(); // Moves the Activity to the RESUMED state
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

Consider using a more descriptive comment to explain what the setup() method does. While it moves the Activity to the RESUMED state, elaborating on the implications or purpose of this state change could be beneficial for readers unfamiliar with the Activity lifecycle.

Suggested change
controller.setup(); // Moves the Activity to the RESUMED state
controller.setup(); // Moves the Activity to the RESUMED state, making it ready for user interaction

Copy link
Member Author

Choose a reason for hiding this comment

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

Not sure if it is relevant to include this. Let me know if you think it makes sense

Copy link
Member

Choose a reason for hiding this comment

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

It's fine to keep the existing version instead of Gemini one.

@MGaetan89 MGaetan89 merged commit f15d3ff into robolectric:master Jul 9, 2025
7 checks passed
@MGaetan89 MGaetan89 deleted the update_readme branch July 9, 2025 06:38
@hoisie
Copy link
Contributor

hoisie commented Jul 9, 2025

Thanks for deleting this. The deckard project is highly obsolete.

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