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

chore: do not override RSTUDIO_VERSION in CI #439

Merged
merged 13 commits into from
May 14, 2024
Merged

Conversation

rokroskar
Copy link
Member

No double-override.

@rokroskar rokroskar requested a review from leafty March 7, 2024 16:44
@rokroskar
Copy link
Member Author

@leafty I can't tell what 52cb8a6 does since most of it is blank space changes... could you not change the indentation?

@leafty
Copy link
Member

leafty commented Mar 8, 2024

@leafty I can't tell what 52cb8a6 does since most of it is blank space changes... could you not change the indentation?

Sorry, I removed two instances of {{ env.RSTUDIO_VERSION }} and format-on-save kicked in.

@leafty
Copy link
Member

leafty commented Mar 27, 2024

@rokroskar I think we should disable the tests until they are fixed and merge this. The images seem to be working (or we need to manually check all of them).

@rokroskar
Copy link
Member Author

@leafty yes sure, that sounds good. I'm not really up to speed on how to fix these tests so some help would be appreciated there...

@leafty
Copy link
Member

leafty commented Mar 27, 2024

tests may be a bit flaky now, but they should ultimately pass

@rokroskar
Copy link
Member Author

Sorry @leafty I missed that you had updated the tests here. Thanks for that! @olevski could you please review when you get a chance?

@rokroskar rokroskar requested a review from olevski April 15, 2024 09:14
@rokroskar rokroskar enabled auto-merge (squash) April 15, 2024 09:44
tests/package.json Show resolved Hide resolved
tests/package.json Outdated Show resolved Hide resolved
tests/package.json Outdated Show resolved Hide resolved
@rokroskar rokroskar force-pushed the rokroskar-patch-2 branch from 9538265 to be7ece5 Compare May 14, 2024 09:36
@rokroskar rokroskar force-pushed the rokroskar-patch-2 branch from be7ece5 to 16b4600 Compare May 14, 2024 09:48
@rokroskar rokroskar disabled auto-merge May 14, 2024 09:52
@rokroskar rokroskar enabled auto-merge (squash) May 14, 2024 09:53
@rokroskar rokroskar requested a review from olevski May 14, 2024 12:03
@rokroskar rokroskar requested a review from leafty May 14, 2024 12:04
@rokroskar rokroskar merged commit 55c03a0 into main May 14, 2024
46 checks passed
@rokroskar rokroskar deleted the rokroskar-patch-2 branch May 14, 2024 14:53
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