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

Add Rider runtime configurations v2 #694

Merged

Conversation

Takoooooo
Copy link
Contributor

@Takoooooo Takoooooo commented May 23, 2024

I've decided to not include the .run configs for iOS,Android and MacCatalyst since they are included by default for me. If that's wouldn't be true for some of our users we can include them later anyway

GitHub Issue (If applicable): closes #676
based on #682

PR Type

  • Feature

What is the current behavior?

We don't provide Rider .run profiles

What is the new behavior?

We have Rider .run profiles

@CLAassistant
Copy link

CLAassistant commented May 23, 2024

CLA assistant check
All committers have signed the CLA.

@Takoooooo Takoooooo marked this pull request as draft May 23, 2024 19:39
@Takoooooo Takoooooo force-pushed the dev/ds/rider-runtime-configs-v2 branch from 2e9f197 to bc3e037 Compare May 23, 2024 21:09
@Takoooooo
Copy link
Contributor Author

Btw,@jeromelaban what did you mean by "Let's also add a readme file to explain what those files are"? Not sure where this file should be located and how should it work

@Takoooooo Takoooooo marked this pull request as ready for review May 23, 2024 21:41
@Takoooooo Takoooooo marked this pull request as draft May 23, 2024 22:14
@Takoooooo Takoooooo marked this pull request as ready for review May 24, 2024 11:37
@Takoooooo
Copy link
Contributor Author

@dansiegel I've tested all configs on my machine, but it would be nice if you could test and review it on your machine too

@Takoooooo
Copy link
Contributor Author

@Takoooooo Takoooooo closed this Jun 5, 2024
@Takoooooo Takoooooo reopened this Jun 6, 2024
@Takoooooo
Copy link
Contributor Author

Keeping the PR until we'll actually implement that feature on our side

@Takoooooo Takoooooo marked this pull request as draft June 6, 2024 13:45
@jeromelaban jeromelaban marked this pull request as ready for review June 14, 2024 17:15
@jeromelaban jeromelaban merged commit cdc5d75 into unoplatform:main Jun 14, 2024
4 of 6 checks passed
@carldebilly
Copy link
Member

❤️❤️❤️

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.

Consider adding .run configurations for unoapp template
5 participants