Skip to content

skip tests requiring auth, remove hardcoded gh install #47

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Jan 27, 2025
Merged

skip tests requiring auth, remove hardcoded gh install #47

merged 6 commits into from
Jan 27, 2025

Conversation

tkfoss
Copy link
Contributor

@tkfoss tkfoss commented Jan 24, 2025

No description provided.

@tkfoss tkfoss requested a review from codegen-team as a code owner January 24, 2025 00:28
@linear Linear
Copy link

linear bot commented Jan 24, 2025

CG-10450 Enable unauthenticated codemod test runs

This requires auth to github.codegen.app:

uv run pytest tests/codemod/test_codemods.py -n auto

context: https://thecodegen.slack.com/archives/C070DA3BNH3/p1737663274013509?thread_ts=1737660890.493439&cid=C070DA3BNH3

Also: it looks like this codemod uses an outdated API? Assignment.right does not exist. Why is this test not failing

https://github.com/codegen-sh/codegen-sdk/blob/e79170cef354d21245bf028c16acc4a7df2[…]onvert_attribute_to_decorator/convert_attribute_to_decorator.py

@tkfoss tkfoss enabled auto-merge (squash) January 27, 2025 15:31
@tkfoss tkfoss merged commit 7baab10 into develop Jan 27, 2025
6 of 12 checks passed
@tkfoss tkfoss deleted the tom-cg-10450-enable-unauthenticated-codemod-test-runs branch January 27, 2025 15:32
tkfoss pushed a commit that referenced this pull request Jan 27, 2025
Revert "skip tests requiring auth, remove hardcoded gh install (#47)"
fb4aa68 
This reverts commit 7baab10.
@tkfoss tkfoss mentioned this pull request Jan 27, 2025
Merged
tkfoss pushed a commit that referenced this pull request Jan 27, 2025
Revert "skip tests requiring auth, remove hardcoded gh install" (#115)
f1a00f9 
Reverts #47
eacodegen pushed a commit that referenced this pull request Jan 27, 2025
skip tests requiring auth, remove hardcoded gh install (#47)
daf57a5
eacodegen pushed a commit that referenced this pull request Jan 27, 2025
Revert "skip tests requiring auth, remove hardcoded gh install" (#115)
071a0e3 
Reverts #47
@tkfoss tkfoss restored the tom-cg-10450-enable-unauthenticated-codemod-test-runs branch January 28, 2025 02:06
tkfoss pushed a commit that referenced this pull request Jan 28, 2025
Revert "skip tests requiring auth, remove hardcoded gh install" (#115)
a47158b 
Reverts #47
jayhack pushed a commit that referenced this pull request Feb 9, 2025
skip tests requiring auth, remove hardcoded gh install (#47)
a69ded9
jayhack pushed a commit that referenced this pull request Feb 9, 2025
Revert "skip tests requiring auth, remove hardcoded gh install" (#115)
07a905c 
Reverts #47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jayhack jayhack jayhack approved these changes

@codegen-team codegen-team Awaiting requested review from codegen-team codegen-team is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tkfoss @jayhack