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

Implemented endpoints for modifying and removing Bundle IDs #60

Closed
wants to merge 7 commits into from

Conversation

ruipfcosta
Copy link
Collaborator

This PR implements the endpoints to modify and remove Bundle IDs. I will add the remaining endpoints in a separate PR (still need to get my head around the implementation 😅).

// Created by Rui Costa on 30/12/2019.
//


Copy link

Choose a reason for hiding this comment

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

⚠️ Limit vertical whitespace to a single empty line. Currently 2.
vertical_whitespace BundleIdUpdateRequest.swift:8

public struct BundleIdUpdateRequest: Encodable {

/// The resource data.
public let data: Data
Copy link

Choose a reason for hiding this comment

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

⚠️ An ‘instance_property’ should not be placed amongst the type content(s) ‘subtype’.
type_contents_order BundleIdUpdateRequest.swift:15

@ghost
Copy link

ghost commented Dec 31, 2019

1 Message
📖 View more details on Bitrise

Generated by 🚫 Danger

public let name: String
}
}

Copy link

Choose a reason for hiding this comment

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

⚠️ Files should have a single trailing newline.
trailing_newline BundleIdUpdateRequest.swift:31

@ruipfcosta
Copy link
Collaborator Author

@AvdLee CI there seems to be some issues with CI. It shows the build as failed for my last commit (cf1b7be), but when I navigate to Bitrise I can see it is still referring to the second to last commit (86fc68f) 🤔

@ruipfcosta
Copy link
Collaborator Author

I realised these endpoints have already been implemented in #57. Closing this PR.

@ruipfcosta ruipfcosta closed this Dec 31, 2019
@ruipfcosta ruipfcosta deleted the bundle-id-endpoints branch December 31, 2019 15:21
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.

1 participant