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 createRepo public endpoint #59

Merged
merged 2 commits into from
Jun 5, 2024
Merged

Add createRepo public endpoint #59

merged 2 commits into from
Jun 5, 2024

Conversation

riya-n
Copy link
Contributor

@riya-n riya-n commented Jun 3, 2024

  • this should create a trunk repo for orgs that don't already have them
  • calls the createRepo public endpoint: https://github.com/trunk-io/trunk/pull/14243
  • tested upload on a test repo and seems to work fine
  • TODO: wait for endpoint to be on prod, then merge this and update cli version on uploader

@riya-n riya-n requested review from zaycev and gnalh June 3, 2024 19:00
@gnalh
Copy link
Contributor

gnalh commented Jun 3, 2024

Is this endpoint available on prod yet?

Copy link

trunk-staging-io bot commented Jun 3, 2024

💊 0 quarantined ✅ 10 passed 🕐 10 new ⋅ (learn more)

docs ⋅ learn more about trunk.io

src/main.rs Show resolved Hide resolved
@@ -56,6 +65,12 @@ pub struct QuarantineBulkTestStatus {
pub quarantine_results: Vec<QuarantineResult>,
}

#[derive(Debug, Serialize, Clone, Deserialize)]
pub struct TrunkRepo {
Copy link
Contributor

Choose a reason for hiding this comment

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

If we aren't using response object, maybe we can have empty schema for now.

Copy link
Contributor Author

@riya-n riya-n Jun 5, 2024

Choose a reason for hiding this comment

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

Yeah, we aren't using it, but we could leave this as a sanity check that the repoId exists in the response

@riya-n riya-n merged commit c14c2a1 into main Jun 5, 2024
5 checks passed
@riya-n riya-n deleted the riya/add-create-repo branch June 5, 2024 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants