Skip to content
This repository was archived by the owner on Dec 3, 2025. It is now read-only.

Conversation

@taylor-cb
Copy link
Contributor

Ticket

Plat-14318
Related tickets:
Related PRs:

Type of PR

  • Bugfix
  • New feature
  • Minor changes

Did you make changes to modules or created a new module?

Changes introduced

Removing CLI from modules repo, as it's now been moved to https://github.com/crowdbotics/cli

@taylor-cb taylor-cb requested a review from danielsousaio April 10, 2024 18:18
Metalheart88
Metalheart88 previously approved these changes Apr 10, 2024
README.md Outdated
Comment on lines 33 to 35
- [python](https://www.python.org/)
- [pipenv](https://pypi.org/project/pipenv/)
- [urllib3](https://urllib3.readthedocs.io/en/stable/) without this the project will not run in ```macOS``` environments
- [urllib3](https://urllib3.readthedocs.io/en/stable/) without this the project will not run in `macOS` environments
Copy link
Collaborator

Choose a reason for hiding this comment

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

Those are cli requirements, we should mention that one must install the cli and its requirements and just link to crowdbotics/cli repo

danielsousaio
danielsousaio previously approved these changes Apr 10, 2024
Copy link
Collaborator

@danielsousaio danielsousaio left a comment

Choose a reason for hiding this comment

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

Looks good, just a minor thing on the readme

@taylor-cb taylor-cb dismissed stale reviews from danielsousaio and Metalheart88 via 31b87a6 April 10, 2024 19:23
Metalheart88
Metalheart88 previously approved these changes Apr 10, 2024
alaisgomes
alaisgomes previously approved these changes May 9, 2025
Metalheart88
Metalheart88 previously approved these changes May 9, 2025
@alaisgomes alaisgomes dismissed stale reviews from Metalheart88 and themself via e6dd3ae May 9, 2025 18:16
alaisgomes
alaisgomes previously approved these changes May 9, 2025
Metalheart88
Metalheart88 previously approved these changes May 9, 2025
@alaisgomes alaisgomes dismissed stale reviews from Metalheart88 and themself via e986ce7 May 9, 2025 18:25
@alaisgomes
Copy link
Contributor

The IOS/Android CI seems like it is testing nothing? It's supposed to test the scaffold, which is a completely repo now: https://github.com/crowdbotics/react-native-scaffold
Likely a leftover code from the migration of the scaffold. We should probably remove in a separate PR.

@alaisgomes alaisgomes merged commit ff3a99a into develop May 9, 2025
2 of 4 checks passed
@alaisgomes alaisgomes deleted the PLAT-14318-remove-cli branch May 9, 2025 18:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants